fix(ARC): ensure that instance value is 1 if ARC chain does not exist…

… yet
postalsys · Dec 19, 2024 · ab4c5e9 · ab4c5e9
1 parent 3c47729
commit ab4c5e9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/arc/index.js b/lib/arc/index.js
@@ -78,6 +78,8 @@ const verifyAS = async (chain, opts) => {
 const signAS = async (chain, entry, signatureData) => {
     let { instance, algorithm, selector, signingDomain, bodyHash, cv, signTime, privateKey } = signatureData;
 
+    instance = instance || 1;
+
     const signAlgo = algorithm?.split('-').shift();
 
     signTime = signTime || new Date();
@@ -497,6 +499,8 @@ const createSeal = async (input, data) => {
         await dkimSigner.finalize();
     }
 
+    seal.i = seal.i || 1;
+
     const authResults = `ARC-Authentication-Results: i=${seal.i}; ${seal.authResults}`;
 
     // Step 2. Calculate ARC-Seal

diff --git a/lib/dkim/dkim-signer.js b/lib/dkim/dkim-signer.js
@@ -266,7 +266,7 @@ class DkimSigner extends MessageParser {
                     {},
                     signatureData,
                     {
-                        instance: this.arc?.instance, // ARC only
+                        instance: this.arc?.instance || 1, // ARC only
                         algorithm,
                         canonicalization: this.getCanonicalization(signatureData).canonicalization,