Merge pull request #42 from vripoche/feat-on-server-error-hook

feat: new onServerError hook triggered on server errors

Author: gregberge

packages/smooth/README.md

@@ -898,6 +898,7 @@ The `modules` option on `"preset-env"` should be kept to `false` otherwise webpa
 - onCreateServer
 - onCreateApolloServerConfig
 - onRenderBody
+- onServerError
 - onCreateBabelConfig
 - onCreateWebpackConfig
 - getContents

packages/smooth/src/plugin/nodeHooks/index.js

@@ -5,4 +5,5 @@ export * from './onCreateBabelConfig'
 export * from './onCreateServer'
 export * from './onCreateWebpackConfig'
 export * from './onRenderBody'
+export * from './onServerError'
 export * from './wrapRootElement'

packages/smooth/src/plugin/nodeHooks/onServerError.js

@@ -0,0 +1,4 @@
+import { applyHook } from '../node'
+
+export const onServerError = config => ({ error, req }) =>
+  applyHook(config, 'onServerError', { error, req })

packages/smooth/src/server/expressApp.js

@@ -5,7 +5,7 @@ import cors from 'cors'
 import { ApolloServer } from 'apollo-server-express'
 import { getContext } from './apollo'
 import ssr from './ssr'
-import { onCreateServer, onCreateApolloServerConfig } from '../plugin/nodeHooks'
+import { onCreateServer, onServerError, onCreateApolloServerConfig } from '../plugin/nodeHooks'
 
 export function createExpressApp({
   dev,
@@ -66,6 +66,11 @@ export function createExpressApp({
   apolloServer.applyMiddleware({ app })
 
   app.use(ssr({ config, schema, fragmentTypes }))
+  
+  app.use((error, req, res, next) => {
+    onServerError(config)({ error, req })
+    next(error)
+  })
 
   if (config.env === 'development') {
     app.use((error, req, res, next) => {