[GitHub] [trafficserver] brbzull0 commented on a diff in pull request #9594: Add support for tunnel route config that has both match group and port variable

["brbzull0 (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org