hnakamur opened a new pull request, #9877: URL: https://github.com/apache/trafficserver/pull/9877
In master and version 9.2.1, content-length header is added when serving the cache with status 204 No Content. It seems caused by [Make 204 cacheable again by maskit · Pull Request #9333 · apache/trafficserver](https://github.com/apache/trafficserver/pull/9333). With this pull request, content-length is not added when serving the cache with status 204 No Content. I made test cases at https://github.com/hnakamur/atstest-go. * The test log *with this fix* is at [log/atsmasterfix-test.log](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/atsmasterfix-test.log) and there is no content-length in the status 204 response from the cache: [log/atsmasterfix-test.log#L35-L41](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/atsmasterfix-test.log#L35-L41). * The test log *without this fix* is at [log/atsmaster-test.log](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/atsmaster-test.log) and there is content-length in the status 204 response from the cache: [log/atsmaster-test.log#L40](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/atsmaster-test.log#L40). (Note aside: the response headers in above logs are sorted alphabetically so the order is not same as the received response). I also made the same fix for trafficserver version 9.2.1 at https://github.com/hnakamur/trafficserver/tree/9_2_1_dont_add_content_length_for_status_204_cache * The test log with this fix is at [log/ats921fix-test.log](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/ats921fix-test.log) and there is no content-length in the status 204 response from the cache: [log/ats921fix-test.log#L35-L41](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/ats921fix-test.log#L35-L41). * The test log without this fix is at [log/ats921-test.log](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/ats921-test.log) and there is content-length in the status 204 response from the cache: [log/ats921-test.log#L40](https://github.com/hnakamur/atstest-go/blob/4c2f9228d05ecfe37486f023503910f6be7e5892/log/ats921-test.log#L40). -- 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]
