Add FailBot.report call to events errors (github#17773)

* Add FailBot.report call to events errors

* Actually throw the error

middleware/events.js

@@ -2,6 +2,7 @@ const express = require('express')
 const { omit } = require('lodash')
 const Ajv = require('ajv')
 const schema = require('../lib/schema-event')
+const FailBot = require('../lib/failbot')
 
 const OMIT_FIELDS = ['type', 'token']
 
@@ -10,15 +11,17 @@ const ajv = new Ajv()
 const router = express.Router()
 
 router.post('/', async (req, res, next) => {
+  const isDev = process.env.NODE_ENV === 'development'
+  const referrer = req.get('referrer')
   const fields = omit(req.body, '_csrf')
 
   if (!ajv.validate(schema, fields)) {
-    if (process.env.NODE_ENV === 'development') console.log(ajv.errorsText())
+    if (isDev) console.log(ajv.errorsText())
     return res.status(400).json({})
   }
 
   // Don't depend on Hydro on local development
-  if (process.env.NODE_ENV === 'development' && !req.hydro.maySend()) {
+  if (isDev && !req.hydro.maySend()) {
     return res.status(200).json({})
   }
 
@@ -27,10 +30,22 @@ router.post('/', async (req, res, next) => {
       req.hydro.schemas[fields.type],
       omit(fields, OMIT_FIELDS)
     )
-    if (!hydroRes.ok) return res.status(502).json({})
+
+    if (!hydroRes.ok) {
+      const err = new Error('Hydro request failed')
+      err.status = hydroRes.status
+      throw err
+    }
+
     return res.status(201).json(fields)
   } catch (err) {
-    if (process.env.NODE_ENV === 'development') console.log(err)
+    if (isDev) console.error(err)
+
+    await FailBot.report(err, {
+      path: req.path,
+      referrer
+    })
+
     return res.status(502).json({})
   }
 })