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/03/04 20:53:05 UTC
[GitHub] [trafficserver] shinrich opened a new pull request #7579: Clean up /r/n between transactions on Http1Client
shinrich opened a new pull request #7579:
URL: https://github.com/apache/trafficserver/pull/7579
Another issue seen while debugging H2 to origin. I'm not entirely sure this should be pushed to master. If it should be, I can add an autest to exercise the issue.
Was tracking down spurious ERR_INVALID_REQ failures being recorded in our access logs. Turns out some of our health checking scripts were adding and extra /r/n or just /n at the end of the health checking GET request. ATS would try to interpret the extra white space and fail resulting in the ERR_INVALID_REQ log entry.
For scripting languages like Perl and Python, it is pretty easy to slap on an extra whitespace. I don't think our scripts send multiple requests on the same connection, so this cleanup is just a matter of keeping confusing cruft out of the logs.
----------------------------------------------------------------
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] shinrich closed pull request #7579: Clean up /r/n between transactions on Http1Client
Posted by GitBox <gi...@apache.org>.
shinrich closed pull request #7579:
URL: https://github.com/apache/trafficserver/pull/7579
----------------------------------------------------------------
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] shinrich commented on pull request #7579: Clean up /r/n between transactions on Http1Client
Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #7579:
URL: https://github.com/apache/trafficserver/pull/7579#issuecomment-790981924
Group consensus. Should just fix the bad clients.
----------------------------------------------------------------
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