fix few details and typos

zk-passport · Aug 3, 2024 · da82935 · da82935
1 parent 7f3b4be
commit da82935
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 66 deletions.
diff --git a/circuits/circuits/register/register_ecdsaWithSHA1Encryption.circom b/circuits/circuits/register/register_ecdsaWithSHA1Encryption.circom
@@ -83,4 +83,5 @@ template Register_ecdsaWithSHA1Encryption(n, k, max_datahashes_bytes, nLevels, s
     signal output commitment <== poseidon_hasher.out;
 }
 
+// We hardcode 7 here for ecdsa_with_SHA1
 component main { public [ attestation_id ] } = Register_ecdsaWithSHA1Encryption(43, 6, 320, 16, 7);
diff --git a/circuits/circuits/register/register_ecdsaWithSHA256Encryption.circom b/circuits/circuits/register/register_ecdsaWithSHA256Encryption.circom
@@ -83,4 +83,5 @@ template Register_ecdsaWithSHA256Encryption(n, k, max_datahashes_bytes, nLevels,
     signal output commitment <== poseidon_hasher.out;
 }
 
+// We hardcode 8 here for ecdsa_with_SHA256
 component main { public [ attestation_id ] } = Register_ecdsaWithSHA256Encryption(43, 6, 320, 16, 8);
diff --git a/circuits/circuits/register/register_sha1WithRSAEncryption_65537.circom b/circuits/circuits/register/register_sha1WithRSAEncryption_65537.circom
@@ -62,5 +62,5 @@ template Register_sha1WithRSAEncryption_65537(n, k, max_datahashes_bytes, nLevel
 
 }
 
-// We hardcode 1 here for sha1WithRSAEncryption_65537
+// We hardcode 3 here for sha1WithRSAEncryption_65537
 component main { public [ attestation_id ] } = Register_sha1WithRSAEncryption_65537(121, 17, 320, 16, 3);
diff --git a/circuits/circuits/register/register_sha256WithRSASSAPSS_65537.circom b/circuits/circuits/register/register_sha256WithRSASSAPSS_65537.circom
@@ -62,5 +62,5 @@ template register_sha256WithRSASSAPSS_65537(n, k, max_datahashes_bytes, nLevels,
 
 }
 
-// We hardcode 1 here for sha256WithRSAEncryption_65537
+// We hardcode 4 here for sha256WithRSASSAPSS_65537
 component main { public [ attestation_id ] } = register_sha256WithRSASSAPSS_65537(64, 32, 320, 16, 4);
diff --git a/circuits/circuits/register/register_sha256WithRSASSAPSS_65537.circom.save b/circuits/circuits/register/register_sha256WithRSASSAPSS_65537.circom.save
diff --git a/circuits/tests/register/register_sha1_rsa.test.ts b/circuits/tests/register/register_sha1_rsa.test.ts
@@ -27,7 +27,7 @@ describe('Register - SHA1 RSA', function () {
           './node_modules/@zk-kit/binary-merkle-root.circom/src',
           './node_modules/circomlib/circuits',
           './node_modules/dmpierre/sha1-circom/circuits',
-          './node_modules/@zk-email/circuits ',
+          './node_modules/@zk-email/circuits',
         ],
       }
     );

diff --git a/common/src/constants/mockPassportData.ts b/common/src/constants/mockPassportData.ts
@@ -47,6 +47,7 @@ export const mockPassportData_sha256_rsa_65537 = {
   ],
   photoBase64: 'iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABjElEQVR42mL8//8/AyUYiBQYmIw3...',
 };
+
 export const mockPassportData_sha1_rsa_65537 = {
   mrz: 'P<FRADUPONT<<ALPHONSE<HUGUES<ALBERT<<<<<<<<<24HB818324FRA0402111M3111115<<<<<<<<<<<<<<02',
   signatureAlgorithm: 'sha1WithRSAEncryption',
@@ -178,7 +179,7 @@ export const mockPassportData_sha384_ecdsa = {
   photoBase64: 'iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABjElEQVR42mL8//8/AyUYiBQYmIw3...',
 };
 
-export const mockPassPortData_sha1_ecdsa = {
+export const mockPassportData_sha1_ecdsa = {
   mrz: 'P<FRADUPONT<<ALPHONSE<HUGUES<ALBERT<<<<<<<<<24HB818324FRA0402111M3111115<<<<<<<<<<<<<<02',
   signatureAlgorithm: 'ecdsa-with-SHA1',
   pubKey: {
@@ -299,7 +300,7 @@ export const mockPassportDatas = [
   mockPassportData_sha256_rsa_65537,
   mockPassportData_sha1_rsa_65537,
   mockPassportData_sha256_rsapss_65537,
-  mockPassPortData_sha1_ecdsa,
+  mockPassportData_sha1_ecdsa,
   mockPassportData_sha256_ecdsa,
   mockPassportData_sha512_ecdsa,
   // mockPassportData_SHA384withECDSA, // temp cause formatting of ecdsa key is not done well now