maskit commented on PR #11942: URL: https://github.com/apache/trafficserver/pull/11942#issuecomment-2591191047
The keyword, "abc", should not appear on the origin server log, because requests that contains the keyword should be blocked by ATS, and the test origin server does not send the keyword. But the keyword accidentally appears because the response data which I added for testing response parser has "abc" where it should be "def". https://github.com/apache/trafficserver/blob/d9195ac85c4958355e5202006d2495989b959652/tests/gold_tests/chunked_encoding/bad_chunked_encoding.test.py#L184-L186 The test passes if I fix the test case (I confirmed this locally), but that makes me wonder why the tests passed on master. -- 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]
