@bedrock/vc-delivery 5.4.0 → 5.5.1

package/lib/oid4/http.js

@@ -12,6 +12,7 @@ import {
 import {asyncHandler} from '@bedrock/express';
 import bodyParser from 'body-parser';
 import cors from 'cors';
+import {logger} from '../logger.js';
 import {UnsecuredJWT} from 'jose';
 import {createValidateMiddleware as validate} from '@bedrock/validation';
 
@@ -156,8 +157,13 @@ export async function createRoutes({
     getConfigMiddleware,
     getExchange,
     asyncHandler(async (req, res) => {
-      const response = await oid4vci.processAccessTokenRequest({req, res});
-      res.json(response);
+      let result;
+      try {
+        result = await oid4vci.processAccessTokenRequest({req, res});
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
+      res.json(result);
     }));
 
   // a credential delivery server endpoint
@@ -195,41 +201,46 @@ export async function createRoutes({
         }
       }
       */
-      const result = await oid4vci.processCredentialRequests({
-        req, res, isBatchRequest: false
-      });
-      if(!result) {
-        // DID proof request response sent
-        return;
-      }
-
-      // send VC(s)
-      const {
-        response: {verifiablePresentation: {verifiableCredential}},
-        format
-      } = result;
-      // FIXME: "format" doesn't seem to be in the spec anymore (draft 14+)...
-      const credentials = verifiableCredential.map(vc => {
-        // parse any enveloped VC
-        let credential;
-        if(vc.type === 'EnvelopedVerifiableCredential' &&
-          vc.id?.startsWith('data:application/jwt,')) {
-          credential = vc.id.slice('data:application/jwt,'.length);
-        } else {
-          credential = vc;
+      let result;
+      try {
+        result = await oid4vci.processCredentialRequests({
+          req, res, isBatchRequest: false
+        });
+        if(!result) {
+          // DID proof request response sent
+          return;
         }
-        return credential;
-      });
 
-      /* Note: The `/credential` route only supports sending VCs of the same
-      type, but there can be more than one of them. The above `isBatchRequest`
-      check will ensure that the workflow used here only allows a single
-      credential request, indicating a single type. */
+        // send VC(s)
+        const {
+          response: {verifiablePresentation: {verifiableCredential}},
+          format
+        } = result;
+        // FIXME: "format" doesn't seem to be in the spec anymore (draft 14+)...
+        const credentials = verifiableCredential.map(vc => {
+          // parse any enveloped VC
+          let credential;
+          if(vc.type === 'EnvelopedVerifiableCredential' &&
+            vc.id?.startsWith('data:application/jwt,')) {
+            credential = vc.id.slice('data:application/jwt,'.length);
+          } else {
+            credential = vc;
+          }
+          return credential;
+        });
+
+        /* Note: The `/credential` route only supports sending VCs of the same
+        type, but there can be more than one of them. The above `isBatchRequest`
+        check will ensure that the workflow used here only allows a single
+        credential request, indicating a single type. */
 
-      // send OID4VCI response
-      const response = credentials.length === 1 ?
-        {format, credential: credentials[0]} : {format, credentials};
-      res.json(response);
+        // send OID4VCI response
+        result = credentials.length === 1 ?
+          {format, credential: credentials[0]} : {format, credentials};
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
+      res.json(result);
     }));
 
   // a credential delivery server endpoint
@@ -240,8 +251,13 @@ export async function createRoutes({
     getConfigMiddleware,
     getExchange,
     asyncHandler(async (req, res) => {
-      const offer = await oid4vci.getCredentialOffer({req});
-      res.json(offer);
+      let result;
+      try {
+        result = await oid4vci.getCredentialOffer({req});
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
+      res.json(result);
     }));
 
   // a batch credential delivery server endpoint
@@ -281,29 +297,37 @@ export async function createRoutes({
         }]
       }
       */
-      const result = await oid4vci.processCredentialRequests({
-        req, res, isBatchRequest: true
-      });
-      if(!result) {
-        // DID proof request response sent
-        return;
-      }
-
-      // send VCs
-      const {response: {verifiablePresentation}, format} = result;
-      // FIXME: "format" doesn't seem to be in the spec anymore (draft 14+)...
-      const responses = verifiablePresentation.verifiableCredential.map(vc => {
-        // parse any enveloped VC
-        let credential;
-        if(vc.type === 'EnvelopedVerifiableCredential' &&
-          vc.id?.startsWith('data:application/jwt,')) {
-          credential = vc.id.slice('data:application/jwt,'.length);
-        } else {
-          credential = vc;
+      let result;
+      try {
+        result = await oid4vci.processCredentialRequests({
+          req, res, isBatchRequest: true
+        });
+        if(!result) {
+          // DID proof request response sent
+          return;
         }
-        return {format, credential};
-      });
-      res.json({credential_responses: responses});
+
+        // send VCs
+        const {
+          response: {verifiablePresentation: {verifiableCredential}},
+          format
+        } = result;
+        // FIXME: "format" doesn't seem to be in the spec anymore (draft 14+)...
+        result = verifiableCredential.map(vc => {
+          // parse any enveloped VC
+          let credential;
+          if(vc.type === 'EnvelopedVerifiableCredential' &&
+            vc.id?.startsWith('data:application/jwt,')) {
+            credential = vc.id.slice('data:application/jwt,'.length);
+          } else {
+            credential = vc;
+          }
+          return {format, credential};
+        });
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
+      res.json({credential_responses: result});
     }));
 
   // an OID4VP verifier endpoint
@@ -315,13 +339,18 @@ export async function createRoutes({
     getConfigMiddleware,
     getExchange,
     asyncHandler(async (req, res) => {
-      const {
-        authorizationRequest
-      } = await oid4vp.getAuthorizationRequest({req});
-      // construct and send authz request as unsecured JWT
-      const jwt = new UnsecuredJWT(authorizationRequest).encode();
-      res.set('content-type', 'application/oauth-authz-req+jwt');
-      res.send(jwt);
+      let result;
+      try {
+        const {
+          authorizationRequest
+        } = await oid4vp.getAuthorizationRequest({req});
+        // construct and send authz request as unsecured JWT
+        result = new UnsecuredJWT(authorizationRequest).encode();
+        res.set('content-type', 'application/oauth-authz-req+jwt');
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
+      res.send(result);
     }));
 
   // an OID4VP verifier endpoint
@@ -335,7 +364,36 @@ export async function createRoutes({
     getConfigMiddleware,
     getExchange,
     asyncHandler(async (req, res) => {
-      const result = await oid4vp.processAuthorizationResponse({req});
+      let result;
+      try {
+        result = await oid4vp.processAuthorizationResponse({req});
+      } catch(error) {
+        return _sendOID4Error({res, error});
+      }
       res.json(result);
     }));
 }
+
+function _sendOID4Error({res, error}) {
+  logger.error(error.message, {error});
+  const status = error.details?.httpStatusCode ?? 500;
+  const oid4Error = {
+    error: _camelToSnakeCase(error.name ?? 'OperationError'),
+    error_description: error.message
+  };
+  if(error?.details?.public) {
+    oid4Error.details = error.details;
+    // expose first level cause only
+    if(oid4Error.cause?.details?.public) {
+      oid4Error.cause = {
+        name: error.cause.name,
+        message: error.cause.message
+      };
+    }
+  }
+  res.status(status).json(oid4Error);
+}
+
+function _camelToSnakeCase(s) {
+  return s.replace(/[A-Z]/g, (c, i) => (i === 0 ? '' : '_') + c.toLowerCase());
+}

package/lib/oid4/oid4vci.js

@@ -552,9 +552,10 @@ async function _requestDidProof({res, exchangeRecord}) {
   const {exchange, meta: {expires}} = exchangeRecord;
   const ttl = Math.floor((expires.getTime() - Date.now()) / 1000);
 
-  res.status(400).json({
-    error: 'invalid_or_missing_proof',
-    error_description:
+  _sendOID4Error({
+    res,
+    error: 'invalid_proof',
+    description:
       'Credential issuer requires proof element in Credential Request',
     // use exchange ID
     c_nonce: exchange.id,
@@ -588,10 +589,19 @@ async function _requestOID4VP({authorizationRequest, res}) {
   challenge to be signed is just the exchange ID itself. An exchange cannot
   be reused and neither can a challenge. */
 
-  res.status(400).json({
+  _sendOID4Error({
+    res,
     error: 'presentation_required',
-    error_description:
+    description:
       'Credential issuer requires presentation before Credential Request',
     authorization_request: authorizationRequest
   });
 }
+
+function _sendOID4Error({res, error, description, status = 400, ...rest}) {
+  res.status(status).json({
+    error,
+    error_description: description,
+    ...rest
+  });
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bedrock/vc-delivery",
-  "version": "5.4.0",
+  "version": "5.5.1",
   "type": "module",
   "description": "Bedrock Verifiable Credential Delivery",
   "main": "./lib/index.js",