enova / sidekiq-rate-limiter

Redis-backed, per-worker rate limits for job processing.
http://rubygems.org/gems/sidekiq-rate-limiter
MIT License
192 stars 43 forks source link

Rate limits with ActiveJob & ActionMailer #28

Open bwillis opened 6 years ago

bwillis commented 6 years ago

Both ActiveJob and ActionMailer queue Sidekiq jobs, but they will be wrapped. This causes all of them to use the same rate limit rules. To help with this, the Sidekiq adapter includes in the payload a "wrapped" attribute pointing to the original class (https://github.com/mperham/sidekiq/issues/2248). It seems this would be a useful improvement to use the wrapper when it's present.

What do you think?