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

Implement rate limit in nginx configuration #413

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

Xpirix
Copy link
Collaborator

@Xpirix Xpirix commented May 29, 2024

This is for #402 according to #402 (comment).

Change summary

  • Apply a rate limit of 10 requests per second for each IP address

@timlinux
Copy link
Member

Is it 10 per second or 10 per minute @Xpirix ?

@Xpirix
Copy link
Collaborator Author

Xpirix commented May 29, 2024

@timlinux This PR proposes a rate limit of 10 requests per second for Nginx. This is for all requests including downloads.
The rate limit of 10 per minute is for the download only with the human validation as proposed in #405 (It also suggests 10 requests per second for Nginx).

@dimasciput dimasciput merged commit ad7aed0 into qgis:master Jun 17, 2024
2 checks passed
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.

3 participants