Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/tmp auth #341

Closed
wants to merge 1 commit into from
Closed

Fix/tmp auth #341

wants to merge 1 commit into from

Conversation

rhanka
Copy link
Member

@rhanka rhanka commented Dec 30, 2022

Moves anonymous temporary session scope from duration to number of requests (search api), moving to a daily static window and keeping ban time:

  • Number of requests is BACKEND_TMP_MAX ans set to 300 by default.
  • If more than 300 request, IP is banned for BACKEND_TMP_DURATION secondes (14400=4 hours by default).
  • number of request are fully reseted in BACKEND_TMP_WINDOW seconds (86400s = 24h) after 1st request (basic non-moving window).
  • User can still register with its login.

Currents stats are :

  • above 2r/j: 3000 users / day, cumulated users 70000 users in a month
  • 50r/j: 650 users/day, 1500 users/month
  • 100r/j: 250 users/day, 6000 users/month
  • 200r/j: 100 users/day; 1500 users/month
  • 300r/j: 50 users/day; 700 users/month

This means current settings (token of 30 days, 300 r/j) with a 50 users threshold.

@rhanka rhanka closed this Dec 30, 2022
@rhanka rhanka deleted the fix/tmp-auth branch December 30, 2022 04:31
@rhanka rhanka restored the fix/tmp-auth branch December 30, 2022 04:31
@cristianpb cristianpb deleted the fix/tmp-auth branch February 13, 2023 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant