appsignal-nodejs
appsignal-nodejs copied to clipboard
Remove the need for an Express error handler
The OpenTelemetry Express instrumentation now automatically records exceptions without the need of an error handler.
See: https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1657
ToDo
- [ ] Update the integration dependency
- [ ] Update test-setups
- [ ] Update the docs
Going by the linked PR, it seems that it only records exceptions in middlewares? Does that cover our needs?
That change doesn't cover our needs.
Here's an open issue reporting problems with the change: https://github.com/open-telemetry/opentelemetry-js-contrib/issues/2146
Internal link with the conversation: https://appsignal.slack.com/archives/C06TU59PXDF/p1714031628760559