[openstreetmap/openstreetmap-website] Remove hardcoded rate limit value. Fixes #4427 (PR #4464)
Tom Hughes
notifications at github.com
Tue Jan 9 18:53:15 UTC 2024
@tomhughes requested changes on this pull request.
> @@ -63,6 +63,7 @@ max_friends_per_hour: 60
min_changeset_comments_per_hour: 1
initial_changeset_comments_per_hour: 6
max_changeset_comments_per_hour: 60
+min_changeset_comments_for_max_rate_limit: 200
That's a very long name and also inaccurate because it's the maximum number of changesets we consider, not the minimum.
I'm not sure what the best name is though - maybe something like `changeset_comments_for_rate_limit` would do? It should also be moved above the other settings I think rather then intermingling it with the `per_hour` settings.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4464#pullrequestreview-1811793712
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4464/review/1811793712 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240109/cc6ca315/attachment.htm>
More information about the rails-dev
mailing list