shinrich commented on pull request #7226: URL: https://github.com/apache/trafficserver/pull/7226#issuecomment-700744008
Seems like a reasonable approach. If the netvc is disabled, I think we still want the changes in PR #7134 so we don't lose the previously set inactivity timeout delta if the netvc is temporarily disabled. But your are correct with the current code, if the netvc is temporarily disabled and the next inactivity timeout is set to 0 and we hit an iteration of the inactivity cop, that netvc's inactivity timeout will be reset to the default inactivity timeout which is probably not desirable since it does reset the previously set inactivity timeout. ---------------------------------------------------------------- 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]
