fix(plugin-koa): Ensure unhandled Koa errors are logged out

packages/plugin-koa/src/koa.js

@@ -33,6 +33,13 @@ module.exports = {
           ctx.bugsnag.notify(createReportFromErr(err, handledState))
         }
         if (!ctx.response.headerSent) ctx.response.status = err.status || 500
+        try {
+          // this function will throw if you give it a non-error, but we still want
+          // to output that, so if it throws, pass it back what it threw (a TypeError)
+          ctx.app.onerror(err)
+        } catch (e) {
+          ctx.app.onerror(e)
+        }
       }
     }
 

packages/plugin-koa/test/koa.test.js

@@ -17,4 +17,22 @@ describe('plugin: koa', () => {
     expect(typeof middleware.errorHandler).toBe('function')
     expect(middleware.errorHandler.length).toBe(2)
   })
+
+  describe('requestHandler', () => {
+    it('should call through to app.onerror to ensure the error is logged out', (done) => {
+      const c = new Client(VALID_NOTIFIER)
+      c.setOptions({ apiKey: 'api_key' })
+      c.configure()
+      c.use(plugin)
+      const middleware = c.getPlugin('koa')
+      const mockCtx = {
+        req: { connection: { address: () => ({ port: 1234 }) } },
+        request: { query: {} },
+        res: {},
+        response: { headerSent: false },
+        app: { onerror: () => done() }
+      }
+      middleware.requestHandler(mockCtx)
+    })
+  })
 })