Skip to content

Comments

docs/nginx: Update 429 resolution documentation#319

Merged
tdgroot merged 1 commit intomasterfrom
update_429_resolve_doc
Jan 15, 2025
Merged

docs/nginx: Update 429 resolution documentation#319
tdgroot merged 1 commit intomasterfrom
update_429_resolve_doc

Conversation

@tdgroot
Copy link
Member

@tdgroot tdgroot commented Jan 14, 2025

The conn_limit_map variable doesn't seem to work very reliable. In our testing, we found that limit_conn_per_ip is much better.

The `conn_limit_map` variable doesn't seem to work very reliable. In our
testing, we found that `limit_conn_per_ip` is much better.
@github-actions
Copy link
Contributor

@tdgroot tdgroot merged commit ec52e73 into master Jan 15, 2025
9 checks passed
@tdgroot tdgroot deleted the update_429_resolve_doc branch January 15, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants