From 5648f3e2cfc0cceb85d60f5a570d8ae21d65bd0f Mon Sep 17 00:00:00 2001 From: Sasha Romijn Date: Mon, 17 Jun 2024 21:26:37 +0200 Subject: [PATCH] Fix #1063 - Update comments to detail user limits in single test --- internetnl/settings.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/internetnl/settings.py b/internetnl/settings.py index 317cf2d61..ce84ccb04 100644 --- a/internetnl/settings.py +++ b/internetnl/settings.py @@ -611,8 +611,11 @@ if DJANGO_IS_PROXIED: SECURE_PROXY_SSL_HEADER = ("HTTP_X_FORWARDED_PROTO", "https") -# Limit the number of tests a client can perform in a while. The exact implementation is to be documented. -# raise the roof of this number to remove this cap. 30 was a limit inherited that comes across as sane. +# Limit the number of tests a client can perform in single test. +# This is the maximum number of tasks that can be run simultaneously per client IP. +# The counter is increased when a task starts, decreased when a task completes, +# and the Redis entry has a 2-hour expiry. Task is an individual category, e.g. RPKI web, +# which means one domain test is 5 tasks. CLIENT_RATE_LIMIT = int(getenv("CLIENT_RATE_LIMIT", 30)) # --- Routinator settings