hapijs/hapi-pino

Is a default failAction needed to log errors?

Opened this issue · 2 comments

rgov commented

I'm completely new to Hapi, I'm just trying to contribute to another project that uses it.

If I am using hapi-pino, is it useful to have a default route.options.response.failAction like the following:

    routes: {
      response: {
        failAction: (request, h, err) => {
          request.log(['error'], err);
          return Boom.badRequest();
        }
      }
    }

Or without this default failAction, will errors still be captured by hapi-pino?

I do not know. I'm not really an Hapi user myself anymore.

IIRC errors are captured even without failAction, e.g. in case of internal server errors, internal accept-encoding errors, or failed requests. In such cases, the event's error object is available via the err property.