Include name of job class in context for Sidekiq and Resque errors.

[johanlunds]

Fixes #258

I thought making context consistent was a good idea. I have not tested the change for Resque. I copied my code from the existing failure handler for Airbrake, so I think it should work. (I did not understand the code for delayed_job - it doesn't seem like it's setting context).

Examples:

• NSQPoster on default
• SendEmailToCustomer on emails

Screenshot

What do you think?

[kattrali]

Thank you for the contribution, @johanlunds. This looks good.

[kattrali]

Looking it over for Resque now.

[johanlunds]

Could potentially change it to be NSQPoster@default instead of NSQPoster on default. That sort of matches movies#create that is the context for (Rails) request errors, and reads pretty well "NSQPoster at (@) default (queue)"

[kattrali]

@johanlunds That makes sense. Would you mind making this change?

[johanlunds]

Sure. Will do

On 12 maj 2016 19:01 +0200, Delisa [email protected], wrote:

@johanlunds(https://github.com/johanlunds)That makes sense. Would you mind making this change?

—
You are receiving this because you were mentioned.
Reply to this email directly orview it on GitHub(#293 (comment))

[codeanpeace]

Any updates on this?

[kattrali]

Thanks for the bump, @codeanpeace, and the PR, @johanlunds. I made the formatting change and released the new context display in v4.2.0

[johanlunds]

Thank you! Sorry for the delay.