shinrich opened a new pull request #7261:
URL: https://github.com/apache/trafficserver/pull/7261
[Step to Reproduce crash]
1. Set the following configurations in records.config.
CONFIG proxy.config.http.attach_server_session_to_client INT 1
CONFIG proxy.config.net.default_inactivity_timeout INT 10
2. Restart ATS
3. Send a GET request with HTTP/2 to ATS that will communicate with an
origin server
4. Wait for inactivity timeout, 10 seconds, that was specified by the config
of default_inactivity_timeout.
5. Crash
The problem was that the
proxy.config.http.attach_server_session_to_client setting only currently
makes sense for HTTP/1. x user agents. As we make our origin connections both
HTTP/2 and HTTP/1.x it may make sense to add this sticky session support
between HTTP/2 user agent and server sessions.
This fix avoids setting the sticky server session if the user agent is
HTTP2. It will instead just close the server session. So the server session
so not left floating around with a reference to the deleted HttpSM..
----------------------------------------------------------------
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]
