fix: added stack logging to getDatabase

forwardemail · Jun 18, 2024 · a75cdc6 · a75cdc6
1 parent 4f90037
commit a75cdc6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/app/models/logs.js b/app/models/logs.js
@@ -304,6 +304,9 @@ Logs.pre('validate', function (next) {
   next();
 });
 
+/*
+// NOTE: this deletes object references for p-retry
+//       we are removing in the interim
 // safeguard for sensitive stuff
 Logs.pre('save', function (next) {
   // delete err.payload.user.password (safeguard)
@@ -316,6 +319,7 @@ Logs.pre('save', function (next) {
 
   next();
 });
+*/
 
 //
 // NOTE: this comes after validate because validation ensures `required: true`

diff --git a/helpers/get-database.js b/helpers/get-database.js
@@ -126,6 +126,7 @@ async function getDatabase(
   newlyCreated = false,
   customDbFilePath = false
 ) {
+  const { stack } = new Error('stack');
   // return early if the session.db was already assigned
   if (
     session.db &&
@@ -566,6 +567,9 @@ async function getDatabase(
         }
       }
     } catch (err) {
+      err._stack = stack;
+      err.session = session;
+
       if (
         err.code !== 'SQLITE_ERROR' ||
         !err.message.startsWith('no such table:')