moonchen commented on PR #13737: URL: https://github.com/apache/trafficserver/pull/13737#issuecomment-5915016405
@bneradt Thanks for pointing at #12198 and #12476. I've redesigned this along those lines, and the PR no longer adds a setting: `proxy.config.net.connections_throttle_exempt_list` is gone. Instead, `proxy.config.http.per_client.connection.exempt_list` now also exempts a client from `proxy.config.net.connections_throttle`. That list is already editable through `TSConnectionLimitExemptListAdd()`, `Remove()` and `Clear()`, the Lua functions, and the `connection_exempt_list` cript, so a large list can be managed from a plugin, and one list covers both client connection limits. Exempt connections still count in `proxy.process.net.connections_currently_open`, and the new gauge `proxy.process.net.per_client.connections_exempt_currently_open` shows how many of them are open. I've updated the title and description to match, and answered the CIDR prefix comment inline (#13756). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
