This commit: 27e5406#diff-230614a2ac854622b8f772e73eea3048R25
upgraded Hapi in master, and converted some of the APM server files. In each defaultErrorHandler function for each of the server/routes/* files, we no longer pass in a reply() method, but instead should return or throw the Boom error. We should:
a) return the Boom error from the transactions.ts default error handler
b) change the others where we throw the error to return instead (or throw in all cases, either one is fine but let's be consistent)
This commit: 27e5406#diff-230614a2ac854622b8f772e73eea3048R25
upgraded Hapi in master, and converted some of the APM server files. In each
defaultErrorHandlerfunction for each of theserver/routes/*files, we no longer pass in areply()method, but instead should return or throw theBoomerror. We should:a) return the Boom error from the transactions.ts default error handler
b) change the others where we throw the error to return instead (or throw in all cases, either one is fine but let's be consistent)