Skip to content

proxy.config.http.per_client.connection.exempt_list.filename #12198

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bneradt
Copy link
Contributor

@bneradt bneradt commented Apr 17, 2025

This implements
proxy.config.http.per_client.connection.exempt_list.filename, a configuration for the user to be able to provide a set of IP addresses that are not counted against
proxy.config.net.per_client.max_connections_in.

@bneradt bneradt added this to the 10.2.0 milestone Apr 17, 2025
@bneradt bneradt requested a review from zwoop April 17, 2025 20:10
@bneradt bneradt self-assigned this Apr 17, 2025
@zwoop
Copy link
Contributor

zwoop commented Apr 17, 2025

allow_ sounds a little strange, exempt_ would be better.

This implements
proxy.config.http.per_client.connection.exempt_list.filename, a
configuration for the user to be able to provide a set of IP addresses
that are not counted against
proxy.config.net.per_client.max_connections_in.
@bneradt bneradt force-pushed the per_client_connection_ignore_list branch from ea155d2 to 8aa79d9 Compare April 17, 2025 22:46
@bryancall bryancall requested a review from maskit April 21, 2025 22:39
@bneradt
Copy link
Contributor Author

bneradt commented Apr 24, 2025

allow_ sounds a little strange, exempt_ would be better.

Agreed. Changed to exempt terminology.

@bneradt bneradt changed the title proxy.config.http.per_client.connection.allow_list.filename proxy.config.http.per_client.connection.exempt_list.filename May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants