[GitHub] [trafficserver] SolidWallOfCode edited a comment on pull request #7188: Update sni outbound policy to allow directly setting the outbound SNI.

[GitBox <gi...@apache.org>]

SolidWallOfCode edited a comment on pull request #7188:
URL: https://github.com/apache/trafficserver/pull/7188#issuecomment-691703001


   My first thought was to have a separate plugin API to do this, but then it would interact in non-obvious ways with this overridable setting. I think this is much better precisely because it collides with the setting. If the outbound SNI is to be set to some literal value, it implicitly disables any keyword used for this, therefore IMHO it's best to make that _explicit_. It can be set to a keyword, or a literal, but never both because both makes no sense.


----------------------------------------------------------------
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:
users@infra.apache.org