SolidWallOfCode commented on a change in pull request #7622:
URL: https://github.com/apache/trafficserver/pull/7622#discussion_r609028512
##########
File path: doc/admin-guide/files/records.config.en.rst
##########
@@ -3717,6 +3717,13 @@ Client-Related Configuration
Enables (``1``) or disables (``0``) TLSv1_3 in the ATS client context. If
not specified, enabled by default
+.. ts:cv:: CONFIG proxy.config.ssl.client.alpn_protocol STRING ""
+
+ Set the alpn string that ATS will send to origin during new connections.
By default no ALPN string will be set.
+ To enable HTTP/2 communication to the origin, set this to "h2,http1.1".
Review comment:
I don't see the benefit of defaulting to "http1.1". To add HTTP/2, it
would still be necessary to specific "http2,http1.1" in the configuration. I
agree with Susan's view that, at least initially, this should be backwards
compatible in that no HTTP/2 is done outbound unless explicitly configured.
That's how HTTP/2 inbound was done - it originally required an explicit
"enable" configuration variable to be set.
--
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]
