Finally had some time to look into this some more. Flipping the sense of the ignore_keep_alive method for Http2 didn't seem to affect the graceful shutdown. Digging in some more made me believe that the ignore_keep_alive method didn't do anything anyway. If it is set to false (don't ignore), then the HttpSM will look at the headers to see if there is a PROXY_CONNECTION or CONNECTION field for the incoming connection. For HTTP2 the header parse will fail if either of these header fields are present.
I think I added these methods early on in the ProxyClient restructuring with the idea that this would simplify processing. But since it turns out that it is not a useful simplification, we should just get rid of the extra method. [ Full content available at: https://github.com/apache/trafficserver/pull/4055 ] This message was relayed via gitbox.apache.org for [email protected]
