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


##########
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:
   yes, if both are present I also suggest adding an error as that seems not 
allowed. It should be either one `proxy_protocol_port` or `inbound_local_port` 
with a combination of the groups.



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