lzx404243 commented on code in PR #9594:
URL: https://github.com/apache/trafficserver/pull/9594#discussion_r1167183659


##########
doc/admin-guide/files/sni.yaml.en.rst:
##########
@@ -176,6 +176,8 @@ tunnel_route              Inbound   Destination as an FQDN 
and port, separated b
                                     Protocol <proxy-protocol>` for more 
information on Proxy Protocol and how it is
                                     configured for |TS|.
 
+                                    Note that the match group number can be 
used in combination with the ``{inbound_local_port}`` and 
``{proxy_protocol_port}`` literal strings.
+

Review Comment:
   Thanks for reviewing this and  good point on handling the case where both 
port variables are specified. I updated the doc to make it more explicit that 
only one can be specified. I also added a check in the code that would emit an 
error message when both are present.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to