@pbvision/fastify-firestore-service 0.0.41 → 0.0.43

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pbvision/fastify-firestore-service",
-  "version": "0.0.41",
+  "version": "0.0.43",
   "description": "Web Framework using Fastify and Firestore ORM",
   "license": "Apache-2.0",
   "type": "module",

package/src/api/api.js

@@ -258,9 +258,13 @@ class API {
    * @private
    */
   __setCORSHeaders (origin, headers) {
-    this.__reply.header('Access-Control-Allow-Origin', origin)
+    this.constructor.setCORSHeadersOnReply(this.__reply, origin, headers)
+  }
+
+  static setCORSHeadersOnReply (reply, origin, headers) {
+    reply.header('Access-Control-Allow-Origin', origin)
     if (headers && headers.length) {
-      this.__reply.header('Access-Control-Allow-Headers', headers.join(', '))
+      reply.header('Access-Control-Allow-Headers', headers.join(', '))
     }
   }
 
@@ -413,7 +417,7 @@ class API {
    * used if these fields don't include sensitive information.
    */
   __trackInputsWithSentry () {
-    const inputs = this.getInputsToTrackWithSentry()
+    const inputs = this.constructor.getInputsToTrackWithSentry(this.req)
     // istanbul ignore else
     if (inputs) {
       this.setSentryContext('inputs', inputs)
@@ -427,11 +431,11 @@ class API {
    * helpful debugging information but should only be
    * used if these fields don't include sensitive information.
    */
-  getInputsToTrackWithSentry () {
+  static getInputsToTrackWithSentry (req) {
     return {
-      ...this.req.query,
-      ...this.req.params,
-      ...this.req.body
+      ...req.query,
+      ...req.params,
+      ...req.body
     }
   }
 
@@ -797,7 +801,20 @@ class API {
       let ret
       try {
         if (req.validationError) {
-          req.__sentry = { context: { validationError: req.validationError } }
+          req.__sentry = { context: {} }
+          const errCopy = { ...req.validationError }
+          errCopy.validation = JSON.stringify(errCopy.validation, null, 4)
+          req.__sentry.context.validationError = errCopy
+          try {
+            req.__sentry.context.inputs = this.getInputsToTrackWithSentry(req)
+          } catch (err) {
+            // istanbul ignore next
+            console.warn('trying to get inputs to track failed', err)
+          }
+          if (this.CORS_ORIGIN) {
+            this.setCORSHeadersOnReply(
+              reply, this.getCORSOrigin(), this.CORS_HEADERS)
+          }
           throw new InvalidInputException(req.validationError)
         }
         ret = await this._callAndHandleRequestDone(reply, async () => {