jpeach commented on PR #9767:
URL: https://github.com/apache/trafficserver/pull/9767#issuecomment-1574549310

   
   > This adds an optional port configuration to the fqdn in sni.yaml. Two 
forms are supported:
   > 
   > ```yaml
   > sni:
   >   # single port
   >   - fqdn: example.com:443
   >   # inclusive range of ports
   >   - fqdn: example.com:443-446
   > ```
   
   I don't think that overloading the `fqdn` field is the right approach. Since 
there is a structured config file here, you can just add a new field, maybe 
calling it `inbound_port_range` or something that's relatively obvious.
   


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