backend-manager 3.2.31 → 3.2.33

package/CHANGELOG.md

@@ -15,6 +15,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - `Security` in case of vulnerabilities.
 
 ---
+## [3.2.32] - 2023-01-30
+### Added
+- Modified `.assistant().errorify()` to have defaults of `log`, `sentry`, and `send` to `false` if not specified to prevent accidental logging and premature sending of errors.
+
 ## [3.2.30] - 2023-01-30
 ### Added
 - Modified `.assistant()` token/key check to use `options.apiKey || data.apiKey`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "backend-manager",
-  "version": "3.2.31",
+  "version": "3.2.33",
   "description": "Quick tools for developing Firebase functions",
   "main": "src/manager/index.js",
   "bin": {

package/src/manager/functions/core/actions/api/admin/backup.js

@@ -19,7 +19,7 @@ Module.prototype.main = function () {
     payload.data.payload.deletionRegex = payload.data.payload.deletionRegex ? powertools.regexify(payload.data.payload.deletionRegex) : payload.data.payload.deletionRegex;
 
     if (!payload.user.roles.admin && assistant.isProduction()) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // https://googleapis.dev/nodejs/firestore/latest/v1.FirestoreAdminClient.html#exportDocuments
@@ -62,7 +62,7 @@ Module.prototype.main = function () {
     .catch(async (e) => {
       await self._setMetaStats(e);
 
-      return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: true}));
+      return reject(assistant.errorify(e, {code: 500}));
     });
 
   });
@@ -97,7 +97,7 @@ Module.prototype._setMetaStats = function (error, meta) {
       metadata: Manager.Metadata().set({tag: 'admin:backup'}),
     }, {merge: true})
     .catch(e => {
-      assistant.errorify(e, {code: 500, sentry: false, send: false, log: true});
+      assistant.errorify(e, {code: 500, log: true});
     })
 
     return resolve();

package/src/manager/functions/core/actions/api/admin/create-post.js

@@ -18,7 +18,7 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
     // Perform checks
     if (!payload.user.roles.admin && !payload.user.roles.blogger) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     const repoInfo = assistant.parseRepo(get(self.Manager.config, 'github.repo_website'));
@@ -45,7 +45,7 @@ Module.prototype.main = function () {
     const finalPost = await poster.create(payload.data).catch(e => e);
 
     if (finalPost instanceof Error) {
-      return reject(assistant.errorify(`Failed to post: ${finalPost}`, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Failed to post: ${finalPost}`, {code: 500}));
     }
 
     // Request indexing
@@ -68,7 +68,7 @@ Module.prototype.main = function () {
       return resolve({data: finalPost});
     })
     .catch((e) => {
-      return reject(assistant.errorify(`Failed to post: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Failed to post: ${e}`, {code: 500}));
     })
 
   });

package/src/manager/functions/core/actions/api/admin/cron.js

@@ -13,12 +13,12 @@ Module.prototype.main = function () {
 
     // Check if the user is an admin
     if (!payload.user.roles.admin && assistant.isProduction()) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // Check if the ID is set
     if (!payload.data.payload.id) {
-      return reject(assistant.errorify(`Missing parameter {id}`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Missing parameter {id}`, {code: 400}));
     }
 
     // Run the cron job
@@ -27,7 +27,7 @@ Module.prototype.main = function () {
       return resolve({data: res});
     })
     .catch(e => {
-      return reject(assistant.errorify(e, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(e, {code: 400}));
     })
   });
 

package/src/manager/functions/core/actions/api/admin/database-read.js

@@ -16,9 +16,9 @@ Module.prototype.main = function () {
 
     // Perform checks
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     } else if (!payload.data.payload.path) {
-      return reject(assistant.errorify(`<path> parameter required`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`<path> parameter required`, {code: 400}));
     }
 
     // Read from Firestore

package/src/manager/functions/core/actions/api/admin/database-write.js

@@ -17,9 +17,9 @@ Module.prototype.main = function () {
 
     // Perform checks
     if (payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     } else if (!payload.data.payload.path) {
-      return reject(assistant.errorify(`<path> parameter required`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`<path> parameter required`, {code: 400}));
     }
 
     // Write to Firestore
@@ -29,7 +29,7 @@ Module.prototype.main = function () {
       return resolve({data: payload.data.payload.document});
     })
     .catch((e) => {
-      return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(e, {code: 500}));
     });
   });
 

package/src/manager/functions/core/actions/api/admin/firestore-query.js

@@ -15,7 +15,7 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
     // Perform checks
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // Run queries
@@ -35,7 +35,7 @@ Module.prototype.main = function () {
         return resolve({data: self.docs});
       })
       .catch((e) => {
-        return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(e, {code: 500}));
       })
 
   });

package/src/manager/functions/core/actions/api/admin/firestore-read.js

@@ -16,9 +16,9 @@ Module.prototype.main = function () {
 
     // Perform checks
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     } else if (!payload.data.payload.path) {
-      return reject(assistant.errorify(`<path> parameter required`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`<path> parameter required`, {code: 400}));
     }
 
     // Read from Firestore
@@ -28,7 +28,7 @@ Module.prototype.main = function () {
       return resolve({data: doc.data()});
     })
     .catch(e => {
-      return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(e, {code: 500}));
     })
   });
 

package/src/manager/functions/core/actions/api/admin/firestore-write.js

@@ -19,9 +19,9 @@ Module.prototype.main = function () {
 
     // Perform checks
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     } else if (!payload.data.payload.path) {
-      return reject(assistant.errorify(`Path parameter required.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Path parameter required.`, {code: 400}));
     }
 
     // Set metadata
@@ -37,7 +37,7 @@ Module.prototype.main = function () {
       return resolve({data: {path: payload.data.payload.path}});
     })
     .catch(e => {
-      return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(e, {code: 500}));
     })
   });
 

package/src/manager/functions/core/actions/api/admin/get-stats.js

@@ -17,7 +17,7 @@ Module.prototype.main = function () {
 
     // Perform checks
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // Get stats
@@ -34,7 +34,7 @@ Module.prototype.main = function () {
         }
 
         if (data instanceof Error) {
-          return reject(assistant.errorify(data, {code: 500, sentry: false, send: false, log: false}));
+          return reject(assistant.errorify(data, {code: 500}));
         }
 
         // Retrieve the stats again after updating
@@ -47,13 +47,13 @@ Module.prototype.main = function () {
 
 
         if (data instanceof Error) {
-          return reject(assistant.errorify(data, {code: 500, sentry: false, send: false, log: false}));
+          return reject(assistant.errorify(data, {code: 500}));
         }
 
         return resolve({data: data})
       })
       .catch(function (e) {
-        return reject(assistant.errorify(`Failed to get: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`Failed to get: ${e}`, {code: 500}));
       })
   });
 

package/src/manager/functions/core/actions/api/admin/payment-processor.js

@@ -15,12 +15,12 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
 
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     const productId = _.get(payload, 'data.payload.payload.details.productIdGlobal');
     if (!productId) {
-      return reject(assistant.errorify(`No productId`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`No productId`, {code: 400}));
     }
     const processorPath = `${process.cwd()}/payment-processors/${productId}.js`
     let processor;
@@ -45,7 +45,7 @@ Module.prototype.main = function () {
       return resolve({data: result});
     })
     .catch(e => {
-      return reject(assistant.errorify(`Payment processor @ "${processorPath}" failed: ${e}`, {code: 400, sentry: true, send: false, log: false}));
+      return reject(assistant.errorify(`Payment processor @ "${processorPath}" failed: ${e}`, {code: 400, sentry: true}));
     })
   });
 

package/src/manager/functions/core/actions/api/admin/send-notification.js

@@ -37,17 +37,17 @@ Module.prototype.main = function () {
       self._notificationPayload.notification.click_action.searchParams.set('cb', new Date().getTime())
       self._notificationPayload.notification.click_action = self._notificationPayload.notification.click_action.toString()
     } catch (e) {
-      assistant.errorify(`Failed to add cb to URL: ${e}`, {code: 500, sentry: false, send: false, log: true});
+      assistant.errorify(`Failed to add cb to URL: ${e}`, {code: 500, log: true});
     }
 
     assistant.log('Resolved notification payload', self._notificationPayload)
 
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     if (!payload.data.payload.title || !payload.data.payload.body) {
-      return reject(assistant.errorify(`Parameters <title> and <body> required`, {code: 400, sentry: true, send: false, log: false}));
+      return reject(assistant.errorify(`Parameters <title> and <body> required`, {code: 400, sentry: true}));
     }
 
     await self.getTokens({tags: false})
@@ -55,7 +55,7 @@ Module.prototype.main = function () {
       return resolve({data: payload.response.data})
     })
     .catch(e => {
-      return reject(assistant.errorify(`Failed to send notification: ${e}`, {code: 400, sentry: true, send: false, log: false}));
+      return reject(assistant.errorify(`Failed to send notification: ${e}`, {code: 400, sentry: true}));
     })
   });
 

package/src/manager/functions/core/actions/api/admin/sync-users.js

@@ -15,7 +15,7 @@ Module.prototype.main = function () {
 
     // If the user is not an admin, reject
     if (!payload.user.roles.admin && assistant.isProduction()) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // Get lastPageToken from meta/stats
@@ -116,7 +116,7 @@ Module.prototype.main = function () {
         return resolve();
       })
       .catch(e => {
-        return reject(assistant.errorify(e, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(e, {code: 500}));
       })
   });
 

package/src/manager/functions/core/actions/api/firebase/get-providers.js

@@ -44,7 +44,7 @@ Module.prototype.main = function () {
     payload.data.payload.firebaseApiKey = payload.data.payload.firebaseApiKey || _.get(Manager, 'config.firebaseConfig.apiKey') || false;
 
     if (!payload.data.payload.firebaseApiKey) {
-      return reject(assistant.errorify(`The firebaseApiKey parameter is required.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`The firebaseApiKey parameter is required.`, {code: 400}));
     }
 
     // Default
@@ -91,7 +91,7 @@ Module.prototype.main = function () {
 
             payload.response.data[provider.name] = result;
           } catch (e) {
-            assistant.errorify(`Error parsing error: ${e}`, {sentry: true, send: false, log: true});
+            assistant.errorify(`Error parsing error: ${e}`, {code: 500, sentry: true});
             payload.response.data[provider.name] = false;
           }
         })
@@ -136,7 +136,7 @@ Module.prototype.main = function () {
 
       })
       .catch(e => {
-        assistant.errorify(`Error getting app data: ${e}`, {sentry: false, send: false, log: true})
+        assistant.errorify(`Error getting app data: ${e}`, {code: 500, log: true})
       })
       .finally(r => {
         return resolve({data: payload.response.data});
@@ -144,7 +144,7 @@ Module.prototype.main = function () {
 
     })
     .catch(e => {
-      return reject(assistant.errorify(`Failed to check providers: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Failed to check providers: ${e}`, {code: 500}));
     })
 
   });

package/src/manager/functions/core/actions/api/general/generate-uuid.js

@@ -20,14 +20,14 @@ Module.prototype.main = function () {
 
     if (payload.data.payload.version === '5') {
       if (!payload.data.payload.name) {
-        return reject(assistant.errorify(`You must provide a name to hash for uuid v5.`, {code: 400, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`You must provide a name to hash for uuid v5.`, {code: 400}));
       } else {
         result = uuid.v5(payload.data.payload.name, payload.data.payload.namespace);
       }
     } else if (payload.data.payload.version === '4') {
       result = uuid.v4();
     } else {
-      return reject(assistant.errorify(`v${payload.data.payload.version} is not a valid version.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`v${payload.data.payload.version} is not a valid version.`, {code: 400}));
     }
 
     assistant.log('UUID Generated', payload.data.payload, result);

package/src/manager/functions/core/actions/api/general/send-email.js

@@ -32,9 +32,9 @@ Module.prototype.main = function () {
     }
 
     if (!payload.data.payload.id) {
-      return reject(assistant.errorify(`Parameter {id} is required.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Parameter {id} is required.`, {code: 400}));
     } else if (!payload.data.payload.email) {
-      return reject(assistant.errorify(`Parameter {email} is required.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Parameter {email} is required.`, {code: 400}));
     }
 
     let emailPayload
@@ -46,7 +46,7 @@ Module.prototype.main = function () {
         script(payload.data.payload, Manager.config),
       );
     } catch (e) {
-      return reject(assistant.errorify(`${payload.data.payload.id} is not a valid email ID.`, {code: 400, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`${payload.data.payload.id} is not a valid email ID.`, {code: 400}));
     }
 
     const storage = Manager.storage({temporary: true});
@@ -70,7 +70,7 @@ Module.prototype.main = function () {
     assistant.log('Storage:', storage.getState()['api:general:send-email']);
 
     if (ipData.count >= emailPayload.spamFilter.ip || emailData.count >= emailPayload.spamFilter.email) {
-      self.assistant.errorify(`Spam filter triggered ip=${ipData.count}, email=${emailData.count}`, {code: 429, sentry: false, send: false, log: true});
+      self.assistant.errorify(`Spam filter triggered ip=${ipData.count}, email=${emailData.count}`, {code: 429, log: true});
 
       return resolve({data: {success: true}});
     }
@@ -100,7 +100,7 @@ Module.prototype.main = function () {
       });
     })
     .catch(e => {
-      return reject(assistant.errorify(`Error sending email: ${e}`, {code: 500, sentry: true, send: false, log: false}));
+      return reject(assistant.errorify(`Error sending email: ${e}`, {code: 500, sentry: true}));
     });
 
   });

package/src/manager/functions/core/actions/api/handler/create-post.js

@@ -15,7 +15,7 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
 
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     payload.response.data = {
@@ -83,7 +83,7 @@ Module.prototype.main = function () {
       .catch(e => e);
 
       if (createdInvoice instanceof Error) {
-        return reject(assistant.errorify(createdInvoice, {code: 400, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(createdInvoice, {code: 400}));
       }
 
       // Send invoice
@@ -104,7 +104,7 @@ Module.prototype.main = function () {
       .catch(e => e);
 
       if (sentInvoice instanceof Error) {
-        return reject(assistant.errorify(sentInvoice, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(sentInvoice, {code: 500}));
       }
 
       payload.response.data.invoice = {

package/src/manager/functions/core/actions/api/special/setup-electron-manager-client.js

@@ -32,11 +32,11 @@ Module.prototype.main = function () {
           signInToken = token;
         })
         .catch(e => {
-          error = assistant.errorify(`Failed to create custom token: ${e}`, {code: 500, sentry: false, send: false, log: false});
+          error = assistant.errorify(`Failed to create custom token: ${e}`, {code: 500});
         })
       })
       .catch(e => {
-        assistant.errorify(`Failed to resolve user: ${e}`, {code: 500, sentry: false, send: false, log: true});
+        assistant.errorify(`Failed to resolve user: ${e}`, {code: 500, log: true});
       })
 
       if (error) {
@@ -93,7 +93,7 @@ Module.prototype.main = function () {
       });
     })
     .catch(e => {
-      return reject(assistant.errorify(`Error fetching app details: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Error fetching app details: ${e}`, {code: 500}));
     })
 
   });

package/src/manager/functions/core/actions/api/template.js

@@ -12,7 +12,7 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
 
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     // self.Api.resolveUser({adminRequired: false})

package/src/manager/functions/core/actions/api/test/create-test-accounts.js

@@ -12,7 +12,7 @@ Module.prototype.main = function () {
   return new Promise(async function(resolve, reject) {
 
     if (!payload.user.roles.admin) {
-      return reject(assistant.errorify(`Admin required.`, {code: 401, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(`Admin required.`, {code: 401}));
     }
 
     assistant.log('User:', payload.user);

package/src/manager/functions/core/actions/api/test/webhook.js

@@ -19,7 +19,7 @@ Module.prototype.main = function () {
     if (payload.data.payload.status >= 200 && payload.data.payload.status <= 299) {
       return resolve({data: payload.data.payload.response, status: payload.data.payload.status});
     } else if (payload.data.payload.status >= 400 && payload.data.payload.status <= 599) {
-      return reject(assistant.errorify(payload.data.payload.response || 'Unknown error message provided', {code: payload.data.payload.status, sentry: false, send: false, log: false}));
+      return reject(assistant.errorify(payload.data.payload.response || 'Unknown error message provided', {code: payload.data.payload.status}));
     }
 
   });

package/src/manager/functions/core/actions/api/user/create-custom-token.js

@@ -19,7 +19,7 @@ Module.prototype.main = function () {
         return resolve({data: {token: token}});
       })
       .catch(e => {
-        return reject(assistant.errorify(`Failed to create custom token: ${e}`, {code: 400, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`Failed to create custom token: ${e}`, {code: 400}));
       })
     })
     .catch(e => {

package/src/manager/functions/core/actions/api/user/delete.js

@@ -22,7 +22,7 @@ Module.prototype.main = function () {
         (user?.plan?.status && user?.plan?.status !== 'cancelled')
         || user?.plan?.payment?.active
       ) {
-        return reject(assistant.errorify(`This account cannot be deleted because it has a paid subscription attached to it. In order to delete the account, you must first cancel the paid subscription.`, {code: 400, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`This account cannot be deleted because it has a paid subscription attached to it. In order to delete the account, you must first cancel the paid subscription.`, {code: 400}));
       }
 
       // Signout of all sessions
@@ -54,7 +54,7 @@ Module.prototype.main = function () {
         return resolve({data: {success: true}});
       })
       .catch(e => {
-        return reject(assistant.errorify(`Failed to delete user: ${e}`, {code: 400, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`Failed to delete user: ${e}`, {code: 400}));
       })
     })
     .catch(e => {

package/src/manager/functions/core/actions/api/user/get-active-sessions.js

@@ -30,7 +30,7 @@ Module.prototype.main = function () {
         return resolve({data: data});
       })
       .catch(e => {
-        return reject(assistant.errorify(`Session query error: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`Session query error: ${e}`, {code: 500}));
       })
 
     })

package/src/manager/functions/core/actions/api/user/regenerate-api-keys.js

@@ -39,7 +39,7 @@ Module.prototype.main = function () {
         return resolve({data: newKeys});
       })
       .catch(e => {
-        return reject(assistant.errorify(`Failed to generate keys: ${e}`, {code: 500, sentry: true, send: false, log: false}));
+        return reject(assistant.errorify(`Failed to generate keys: ${e}`, {code: 500, sentry: true}));
       })
 
     })

package/src/manager/functions/core/actions/api/user/sign-out-all-sessions.js

@@ -37,10 +37,10 @@ Module.prototype.main = function () {
             return resolve({data: {sessions: count, message: `Successfully signed ${uid} out of all sessions`}});
           })
           .catch(e => {
-            return reject(assistant.errorify(`Failed to sign out of all sessions: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+            return reject(assistant.errorify(`Failed to sign out of all sessions: ${e}`, {code: 500}));
           })
       } catch (e) {
-        return reject(assistant.errorify(`Failed to sign out of all sessions: ${e}`, {code: 500, sentry: false, send: false, log: false}));
+        return reject(assistant.errorify(`Failed to sign out of all sessions: ${e}`, {code: 500}));
       }
     })
     .catch(e => {
@@ -108,7 +108,7 @@ Module.prototype.signOutOfSession = function (uid, session) {
 
       })
       .catch(e => {
-        assistant.errorify(`Session query error for session ${session}: ${e}`, {code: 500, sentry: true, send: false, log: true});
+        assistant.errorify(`Session query error for session ${session}: ${e}`, {code: 500, sentry: true});
 
         return reject(e);
       })