[GitHub] [trafficserver] shinrich commented on issue #1663: Make proxy.config.http2.accept_no_activity_timeout and proxy.config.http2.no_activity_timeout_in overridable/reloadable

[GitBox <gi...@apache.org>]

shinrich commented on issue #1663:
URL: https://github.com/apache/trafficserver/issues/1663#issuecomment-788002436


   I don't see how the http2 versions of these configs could be overridable.  The documents mark them as overridable.  That should be corrected.
   
   proxy.config.http2.accept_no_activity_timeout is a characteristic of the session.  It tracks how long you will keep the connection open if there is no activity on the connection.  This means no activity on any of the potentially multiple streams(transactions) on the session.
   
   The description for proxy.config.http2.no_activity_timeout_in  seems to be the same.  Need to did into the implementation and update that as well.  But in any case, it is session/connection oriented.  The Http no_transaction_activity timeouts are transaction oriented and apply to transactions that originate from Http2 sessions as well as Http1 sessions.  Thus it makes sense for them to allow per transaction overrides.
   


----------------------------------------------------------------
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:
users@infra.apache.org