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/29 15:31:47 UTC

[GitHub] [trafficserver] shinrich opened a new pull request #8012: Add option to mostly strictly check URL characters

shinrich opened a new pull request #8012:
URL: https://github.com/apache/trafficserver/pull/8012


   Augments proxy.config.http.strict_uri_parsing to take three values rather than just two.  Adding a mostly strict URI checking mode that will not enforce the full RFC restrictions but will ensure that white space and unprintable characters are not allowed.  This restricting will be enough to detect other problems in the request line result in odd URI's based on how ATS parses the request line.
   
   Updates docs, unit test, and autest.  Does not change the default value of proxy.config.http.strict_uri_parsing but may want to consider making 2 the default on the next major release.


-- 
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] shinrich merged pull request #8012: Add option to mostly strictly check URL characters

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


   


-- 
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] ezelkow1 commented on pull request #8012: Add option to mostly strictly check URL characters

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


   Can we get a backport to 8.1.x for 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.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

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



[GitHub] [trafficserver] shinrich commented on pull request #8012: Add option to mostly strictly check URL characters

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


   [approve ci autest]


-- 
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] zwoop commented on pull request #8012: Add option to mostly strictly check URL characters

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


   Cherry-picked to v9.1.x branch.


-- 
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] ezelkow1 removed a comment on pull request #8012: Add option to mostly strictly check URL characters

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


   Can we get a backport to 8.1.x for 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.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

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