Secp256k1 disable malleability check when verifying

TBD54566975 · Oct 17, 2023 · 16556f1 · 16556f1
1 parent f3c7184
commit 16556f1
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/packages/crypto/src/crypto-primitives/secp256k1.ts b/packages/crypto/src/crypto-primitives/secp256k1.ts
@@ -311,9 +311,12 @@ export class Secp256k1 {
     const hashFunction = this.hashAlgorithms[hash];
     const digest = hashFunction(data);
 
-    // Verify operation.
-    const isValid = secp256k1.verify(signature, digest, key);
+    // Verify operation with malleability check disabled. Guaranteed support for low-s
+    // signatures across languages is unlikely especially in the context of SSI. 
+    // Notable Cloud KMS providers do not natively support it either. 
+    // low-s signatures are a requirement for Bitcoin
+    const isValid = secp256k1.verify(signature, digest, key, { lowS: false });
 
     return isValid;
   }
-}
+}