Github user robguima commented on the pull request:
https://github.com/apache/trafficserver/pull/455#issuecomment-180907419
Yes, we can be fairly sure it doesn't (break anything), because:
a) we've added functionality not changing the way it worked before (bug
aside). So, if content-type, md5, and matrix params are not present, it should
behave as before.
b) These fields not being present is actually the most common situation -
so we've actually verified that the old functionality remains as is, because in
most cases, md5, content-type, and matrix params are not used in GET requests.
The only caveat I can think of is using virtual host, which I am not and
can't (very easily). Based on the code though, that little logic (prepending
the host) seems unchanged by this PR.
This code is deployed in "production" right now in our aws region, and
being tested routinely.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
