Skip to content

Conversation

@janklimo
Copy link
Contributor

Changes

  1. I'd like to propose to perform a deep merge inside annotate_exception so no existing context is lost by accident.
  2. Added a spec to make sure the Sidekiq error handler does perform a deep merge on the existing context.

@janklimo janklimo force-pushed the annotate-exception-deep-merge branch from 5aa929c to ab54f8a Compare October 21, 2017 07:03
@nateberkopec
Copy link
Contributor

Simple and reasonable. Thanks!

@nateberkopec nateberkopec merged commit d453544 into getsentry:master Oct 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants