Node.js: Support for breadcrumbs
See original GitHub issueDescription
It’d be awesome to have official support for Breadcrumbs on Node.js.
Describe the solution you’d like
If I’m not mistaken, it should be as simple as removing some lines of code from the @bugsnag/node
package.
Describe alternatives you’ve considered Being sad - but I didn’t like that option… so I hope that’s not the route you take. 😂
Additional context
I wasn’t even aware that it was disabled until recently - as I was under the impression that it had been supported for years now. At work we use an in-house framework for our microservices, that has a bundled version of Bugsnag built-in. The way this framework makes use of the @bugsnag/js
package meant that we avoided creating a client that had the leaveBreadcrumb
method disabled (due to the way the clone client function works). It was a pure coincidence that we avoided this limitation when integrating Bugsnag in our framework.
For the past two years (almost to the date), in production, we’ve been making use of breadcrumbs on Node.js. To ensure that the breadcrumbs contained useful information, upon each request we enter a Node.js domain, and attach a new Bugsnag client to it. This means that no matter where (within that async call stack) we throw/catch an error, we have the ability to send it off to Bugsnag using the same client that was present at the beginning of the request. We override the console logger to look for this Bugsnag client and automatically leave console-type breadcrumbs. This has given us many many years of happy debugging and has no doubt saved countless hours of engineer’s time.
I hope that the fact we’ve been running this in production for the past 2 years, in 39 services, is reason enough to count breadcrumbs a viable feature to (in my opinion) very easily enable - by removing a few lines of code.
Issue Analytics
- State:
- Created 3 years ago
- Comments:7 (3 by maintainers)
Top GitHub Comments
👋 @jmshal
Thanks for the additional context. The lack of breadcrumb support for Node.js is something we’re hoping to revisit when priorities allow 👍
I don’t really understand why this hasn’t been added already, tbh. We’ve (at my work) been using breadcrumbs with the Node.js client for over 3 years now. Will you accept PRs, @yousif-bugsnag? I wouldn’t be able to use the client without breadcrumbs - I am surprised more people aren’t voicing their frustration.