@defra-fish/connectors-lib 1.64.0 → 1.65.0-airbrake-debugging-2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/connectors-lib",
-  "version": "1.64.0",
+  "version": "1.65.0-airbrake-debugging-2",
   "description": "Shared connectors",
   "type": "module",
   "engines": {
@@ -45,6 +45,5 @@
     "ioredis": "4.28.5",
     "node-fetch": "2.7.0",
     "redlock": "4.2.0"
-  },
-  "gitHead": "d6a3d30c7a20169fdf5afb1a93eb1d411fe96abf"
+  }
 }

package/src/airbrake.js

@@ -1,5 +1,7 @@
+import db from 'debug'
 import { Notifier } from '@airbrake/node'
 import { formatWithOptions, inspect } from 'util'
+const debug = db('connectors:airbrake')
 const INSPECT_OPTS = { depth: null, maxStringLength: null, maxArrayLength: null, breakLength: null, compact: true, showHidden: true }
 
 let airbrake = null
@@ -25,15 +27,18 @@ export const initialise = () => {
       errorNotifications: true,
       remoteConfig: false
     })
+    console.log(`created new Notifier with error notifications set ${airbrake._opt.errorNotifications ? 'on' : 'off'}`)
 
     // Proxy the console.warn and console.error methods, notifying airbrake/errbit asynchronously
     const nativeConsoleMethods = {}
     ;['warn', 'error'].forEach(method => {
+      console.log(`replacing console.${method} function`)
       nativeConsoleMethods[method] = console[method].bind(console)
       console[method] = (...args) => {
         const error = args.find(arg => arg instanceof Error) ?? new Error(formatWithOptions(INSPECT_OPTS, ...args))
         const request = args.find(arg => Object.prototype.hasOwnProperty.call(arg, 'headers'))
-        airbrake.notify({
+        console.log(`notifying airbrake for console.${method}, error notifications is ${airbrake._opt.errorNotifications ? 'on' : 'off'}`, INSPECT_OPTS)
+        const res = airbrake.notify({
           error,
           params: { consoleInvocationDetails: { method, arguments: { ...args.map(arg => inspect(arg, INSPECT_OPTS)) } } },
           environment: {
@@ -46,6 +51,7 @@ export const initialise = () => {
             ...(request?.headers?.['user-agent'] && { userAgent: request?.headers?.['user-agent'] })
           }
         })
+        console.log(`Notify result: ${JSON.stringify(res)}`, res.id, res.url, res.error)
         nativeConsoleMethods[method](...args)
       }
     })
@@ -55,6 +61,8 @@ export const initialise = () => {
 
     // Override the @airbrake/node uncaughtException/unhandledRejection handlers with our own as errors were not flushing correctly.
     const flushAndDie = async () => {
+      console.log('Airbrake connector flushing and dying...')
+      debug('Airbrake connector flushing and dying...')
       await airbrake.flush()
       process.exit(1)
     }
@@ -71,6 +79,8 @@ export const initialise = () => {
  */
 export const flush = async () => {
   if (initialise()) {
+    console.log('Airbrake connector flushing...')
+    debug('Airbrake connector flushing...')
     await airbrake.flush()
     airbrake.close()
   }