You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/06/21 18:53:57 UTC
[GitHub] [trafficserver] SolidWallOfCode opened a new pull request #7967: Tweak QUIC protocol stack support to include TLS.
SolidWallOfCode opened a new pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967
QUIC is based on TLS and from an external point of view, it's TLS (i.e. all standard TLS operations are expected to work, e.g. pulling the SANS from the client certificate). Therefore the protocol stack should indicate this.
--
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
[GitHub] [trafficserver] maskit commented on pull request #7967: Tweak QUIC protocol stack support to include TLS.
Posted by GitBox <gi...@apache.org>.
maskit commented on pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967#issuecomment-865450537
I'm not sure if we should do this. If you need to know whether a connection uses TLS, you should call `TSVConnIsSsl`, although it hasn't supported QUIC connection yet. Checking it by string comparison doesn't seem like a good way.
--
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
[GitHub] [trafficserver] maskit commented on pull request #7967: Tweak QUIC protocol stack support to include TLS.
Posted by GitBox <gi...@apache.org>.
maskit commented on pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967#issuecomment-865450537
I'm not sure if we should do this. If you need to know whether a connection uses TLS, you should call `TSVConnIsSsl`, although it hasn't supported QUIC connection yet. Checking it by string comparison doesn't seem like a good way.
--
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
[GitHub] [trafficserver] maskit commented on pull request #7967: Tweak QUIC protocol stack support to include TLS.
Posted by GitBox <gi...@apache.org>.
maskit commented on pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967#issuecomment-868747122
For those who are not familiar with QUIC, this is from Wireshark. TLS is inside QUIC.
<img width="500" alt="image" src="https://user-images.githubusercontent.com/153144/123467947-15f1b800-d62c-11eb-89f6-c26561653458.png">
--
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
[GitHub] [trafficserver] bryancall commented on pull request #7967: Tweak QUIC protocol stack support to include TLS.
Posted by GitBox <gi...@apache.org>.
bryancall commented on pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967#issuecomment-868735934
@rob05c Said he is going to review it.
--
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
[GitHub] [trafficserver] SolidWallOfCode merged pull request #7967: Tweak QUIC protocol stack support to include TLS.
Posted by GitBox <gi...@apache.org>.
SolidWallOfCode merged pull request #7967:
URL: https://github.com/apache/trafficserver/pull/7967
--
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