fix: credential mapper for jtw

Sphereon-Opensource · Feb 18, 2023 · f04345b · f04345b
1 parent 559efe2
commit f04345b
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 19 deletions.
diff --git a/packages/did-auth-siop-op-authenticator/src/session/OpSession.ts b/packages/did-auth-siop-op-authenticator/src/session/OpSession.ts
@@ -72,7 +72,7 @@ export class OpSession {
         supportedDidMethods: this.supportedDidMethods || [],
         expiresIn: this.expiresIn || 6000,
         presentationSignCallback,
-        wellknownDidVerifyCallback
+        wellknownDidVerifyCallback,
       },
       this.context
     )

diff --git a/packages/ssi-types/src/mapper/credential-mapper.ts b/packages/ssi-types/src/mapper/credential-mapper.ts
@@ -274,20 +274,30 @@ export class CredentialMapper {
   }
 
   static toExternalVerifiableCredential(verifiableCredential: any): IVerifiableCredential {
-    if (!verifiableCredential.proof.type) {
-      throw new Error('Verifiable credential proof is missing a type')
-    }
+    let proof
+    if (verifiableCredential.proof) {
+      if (!verifiableCredential.proof.type) {
+        throw new Error('Verifiable credential proof is missing a type')
+      }
 
-    if (!verifiableCredential.proof.created) {
-      throw new Error('Verifiable credential proof is missing a created date')
-    }
+      if (!verifiableCredential.proof.created) {
+        throw new Error('Verifiable credential proof is missing a created date')
+      }
 
-    if (!verifiableCredential.proof.proofPurpose) {
-      throw new Error('Verifiable credential proof is missing a proof purpose')
-    }
+      if (!verifiableCredential.proof.proofPurpose) {
+        throw new Error('Verifiable credential proof is missing a proof purpose')
+      }
 
-    if (!verifiableCredential.proof.verificationMethod) {
-      throw new Error('Verifiable credential proof is missing a verification method')
+      if (!verifiableCredential.proof.verificationMethod) {
+        throw new Error('Verifiable credential proof is missing a verification method')
+      }
+      proof = {
+        ...verifiableCredential.proof,
+        type: verifiableCredential.proof.type,
+        created: verifiableCredential.proof.created,
+        proofPurpose: verifiableCredential.proof.proofPurpose,
+        verificationMethod: verifiableCredential.proof.verificationMethod,
+      }
     }
 
     return {
@@ -297,13 +307,7 @@ export class CredentialMapper {
           ? [verifiableCredential.type]
           : verifiableCredential.type
         : ['VerifiableCredential'],
-      proof: {
-        ...verifiableCredential.proof,
-        type: verifiableCredential.proof.type,
-        created: verifiableCredential.proof.created,
-        proofPurpose: verifiableCredential.proof.proofPurpose,
-        verificationMethod: verifiableCredential.proof.verificationMethod,
-      },
+      proof,
     }
   }
 }