I'm currently saving error properties under the extra
portion of the error type manually, however it would be convenient if this was supported in raven-js
the same way it is done in raven-node
:
https://github.com/getsentry/raven-node/blob/master/lib/parsers.js#L30-L39
Wow, I didn't even realize raven-node did that. Hmm ... yeah, I think we can make the same change.
Some more motivation from my end -- there are a few places in our codebase where we throw new Error('msg for sentry')
with the intention that this will both notify the user that something went wrong as well as log the message and stack trace to sentry.
Cases where we hard error tend to be the most unexpected, so Ideally we could add some extra data to the sentry logs. EG in a default branch of a switch statement, if that branch was unexpected, we would like to hard error and attach the data that we were not expecting as well as any other surrounding useful context.
Even if in most the extra
param is simple, stringify-ing them in the error message is not ideal since then the errors would not de-dupe.
Please see conversation in #483 – totally missed this issue earlier, sorry.
Most helpful comment
Wow, I didn't even realize raven-node did that. Hmm ... yeah, I think we can make the same change.