You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Sidekiq::Limiter::Window has attr_reader :interval, :count, :bucket and Sidekiq::Limiter::Bucket has attr_reader :interval, :count.
I propose adding attr_reader for the configuration options to the limiters that don't have it. I could specifically use it on Sidekiq::Limiter::LeakyBucket, which doesn't have any attribute readers. It would be nice to expose the size and interval.
My use case is that I'm reading the limiter when hitting OverLimit error and using the limiter config details to calculate an optimal backoff time (see discussion in #6141)
The text was updated successfully, but these errors were encountered:
Sidekiq::Limiter::Window
hasattr_reader :interval, :count, :bucket
andSidekiq::Limiter::Bucket
hasattr_reader :interval, :count
.I propose adding
attr_reader
for the configuration options to the limiters that don't have it. I could specifically use it onSidekiq::Limiter::LeakyBucket
, which doesn't have any attribute readers. It would be nice to expose thesize
andinterval
.My use case is that I'm reading the limiter when hitting
OverLimit
error and using the limiter config details to calculate an optimal backoff time (see discussion in #6141)The text was updated successfully, but these errors were encountered: