ezelkow1 opened a new pull request, #11599: URL: https://github.com/apache/trafficserver/pull/11599
Added 2 tests, one with a bad path param and one with a properly encoded path param. ATS returns 403 and 200 properly Added path param checks for success and failure, shuffled tests for correct logging output If there is a spurious path param in a request to urlsig it's going to 403 because it will not match the signature provided. So that should be why it is ok to remove the ParamsSet here since if there are no path params then the new path does not need to be set and params dont need to be updated. If there are path params, and they are good, then the new path will be set which includes the params. If there are path params but they are bad then it is going to 403 anyway -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
