shinrich opened a new pull request #7134: URL: https://github.com/apache/trafficserver/pull/7134
Saw this when trying to override the inactivity timeouts for a tunnel_route. We could override the proxy.config.http.transaction_no_activity_timeout_in and proxy.config.http.transaction_no_activity_timeout_out to 600 seconds (from our default of 30 and 90). But our test stalled tunnel timed out at 300 seconds because our proxy.config.net.default_inactivity_timeout was set to 300, and the inactivity cop resets the timeout to proxy.config.net.default_inactivity_timeout if the next_inactivity_timeout is set to 0. However, for the tunnel case, there is not an opportunity to reset the timeout to the original value. With this code change, the tunnel timeout works as expected without tripping up on the default_inactivity_timeout. This chance also does not seem to impact regular traffic. I am not sure why we are clearing the inactivity timeout if the netvc is both read and write disabled. Looking through the git blames it seems that this logic has been present since the code moved into github in 2009. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
