search

tosdr / edit.tosdr.org

👍👎 A new web app to rate services
https://edit.tosdr.org
GNU Affero General Public License v3.0
215 stars 37 forks source link

remove rate limiting for curators #929

Closed michielbdejong closed 3 years ago

michielbdejong commented 4 years ago

if the logged-in user is a curator, we probably shouldn't need to rate limit them

madoleary commented 3 years ago

Leaving this here as a potential solution to explore : https://github.com/rack/rack-attack/issues/397

michielbdejong commented 3 years ago

Tried changing req.ip to req.ip unless current_user.curator? in each of the throttle rules in config/initializers/rack_attack.rb but current_user is not available there.

michielbdejong commented 3 years ago

As a work-around for now I'll temporarily loosen the update-document limit for everyone for now, because that's the one we will be hitting while reviewing old pending points and old doc crawls.