immobiliare/fastify-sentry

Improve integration with sdk

Closed this issue ยท 2 comments

dnlup commented

Looking at how Sentry is integrated in Express (https://github.com/getsentry/sentry-javascript/blob/master/packages/node/src/handlers.ts) I think we can pick up a thing or two and apply them here, for example:

  • use a function (shouldHandlerError in the Express integration) to customize the event filtering instead of our array of statusCodes
  • improve how we enrich the event data

Implementing these changes would also resolve issues #5 and #3

๐ŸŽ‰ This issue has been resolved in version 5.0.0-next.1 ๐ŸŽ‰

The release is available on:

Your semantic-release bot ๐Ÿ“ฆ๐Ÿš€

๐ŸŽ‰ This issue has been resolved in version 5.0.0 ๐ŸŽ‰

The release is available on:

Your semantic-release bot ๐Ÿ“ฆ๐Ÿš€