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 2020/07/07 14:27:49 UTC

[GitHub] [trafficserver] willwendorfvm opened a new pull request #6976: Use print mode with length in validate_sni debug

willwendorfvm opened a new pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976


   std::string_view is not necessarily null terminated, so a print mode relying on a length instead of null termination should be used here.


----------------------------------------------------------------
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] randall commented on pull request #6976: Use print mode with length in validate_sni debug

Posted by GitBox <gi...@apache.org>.
randall commented on pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976#issuecomment-655106518


   [approve ci]


----------------------------------------------------------------
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] randall commented on pull request #6976: Use print mode with length in validate_sni debug

Posted by GitBox <gi...@apache.org>.
randall commented on pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976#issuecomment-655176409


   [approve ci]


----------------------------------------------------------------
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] randall removed a comment on pull request #6976: Use print mode with length in validate_sni debug

Posted by GitBox <gi...@apache.org>.
randall removed a comment on pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976#issuecomment-655176409






----------------------------------------------------------------
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] SolidWallOfCode merged pull request #6976: Use print mode with length in validate_sni debug

Posted by GitBox <gi...@apache.org>.
SolidWallOfCode merged pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976


   


----------------------------------------------------------------
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] willwendorfvm commented on pull request #6976: Use print mode with length in validate_sni debug

Posted by GitBox <gi...@apache.org>.
willwendorfvm commented on pull request #6976:
URL: https://github.com/apache/trafficserver/pull/6976#issuecomment-655131301


   Fixed clang-format issue.


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