Add allow list for keys (properties) to include in errors.

digitalbazaar · Aug 25, 2024 · cca1204 · cca1204
1 parent 6547df8
commit cca1204
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # bedrock-vc-delivery ChangeLog
 
+## 5.3.2 - 2024-08-dd
+
+### Fixed
+- Allow list specific error keys to include in last error.
+
 ## 5.3.1 - 2024-08-24
 
 ### Fixed

diff --git a/lib/helpers.js b/lib/helpers.js
@@ -13,6 +13,11 @@ import {ZcapClient} from '@digitalbazaar/ezcap';
 
 const {config, util: {BedrockError}} = bedrock;
 
+const ALLOWED_ERROR_KEYS = [
+  'message', 'name', 'type', 'data', 'errors', 'error', 'details', 'cause',
+  'status'
+];
+
 export async function evaluateTemplate({
   workflow, exchange, typedTemplate
 } = {}) {
@@ -106,8 +111,14 @@ export function decodeLocalId({localId} = {}) {
 }
 
 export function stripStacktrace(error) {
-  error = serializeError(error);
-  delete error.stack;
+  // serialize error and allow-list specific properties
+  const serialized = serializeError(error);
+  error = {};
+  for(const key of ALLOWED_ERROR_KEYS) {
+    if(serialized[key] !== undefined) {
+      error[key] = serialized[key];
+    }
+  }
   if(error.errors) {
     error.errors = error.errors.map(stripStacktrace);
   }