Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEPRECATION: Sidekiq exception handlers now take three arguments #1131

Closed
devaniljr opened this issue Oct 4, 2023 · 3 comments · Fixed by #1136
Closed

DEPRECATION: Sidekiq exception handlers now take three arguments #1131

devaniljr opened this issue Oct 4, 2023 · 3 comments · Fixed by #1136

Comments

@devaniljr
Copy link

worker | 2023-10-04T10:02:00.510Z pid=253522 tid=4yc6 INFO: DEPRECATION: Sidekiq exception handlers now take three arguments, see #<Proc:0x00007f07be656108 /home/devanil/.asdf/installs/ruby/3.2.2/lib/ruby/gems/3.2.0/gems/rollbar-3.4.0/lib/rollbar/plugins/sidekiq.rb:15>
worker | 2023-10-04T10:02:00.510Z pid=253522 tid=4y76 ERROR: Error fetching job: Waited 3.0 seconds

      config.error_handlers << proc do |e, context|
        Rollbar::Sidekiq.handle_exception(context, e)
      end
@bf4
Copy link
Contributor

bf4 commented Nov 16, 2023

I'm always surprised by the lack of Ruby support for this product we pay for.. when simple maintenance tasks like this grow stale, I begin to wonder if we should switch to something else

@waltjones is Ruby still supported?

@ghost
Copy link

ghost commented Nov 16, 2023

Hey @devaniljr @bf4, We opened an internal bug ticket when this issue was reported (131105), and it's in our backlog. I'm sorry, this issue wasn't updated at the time. I confirm that Ruby is still a supported SDK of Rollbar, and we plan to fix this issue. I'll get back to you when we have a timeline ready.

@ghost
Copy link

ghost commented Nov 16, 2023

Hey @devaniljr @bf4, We scheduled working on the Ruby SDK after Thanksgiving. Our Ruby dev is currently headed down on another crucial initiative, but I want to assure you we still remember this one. I was informed that the Sidekiq update will probably require a breaking change; hence expect a major version bump.

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 a pull request may close this issue.

2 participants