bneradt opened a new pull request, #12661: URL: https://github.com/apache/trafficserver/pull/12661
The TSHttpHdrPrint function already appends the final CRLF sequence (\r\n\r\n) to terminate HTTP headers, as confirmed by tracing through the implementation (InkAPI.cc -> HTTPHdr::print -> http_hdr_print -> mime_hdr_print). Several plugins were incorrectly adding an additional TSIOBufferWrite with \r\n after calling TSHttpHdrPrint, which resulted in malformed HTTP requests with an extra blank line. On keep-alive connections, this extra CRLF would be interpreted as the start of a second request, causing ATS to log ERR_INVALID_REQ errors when it attempted to parse the incomplete request. This fix removes the unnecessary TSIOBufferWrite calls from the stale_response, background_fetch, cache_fill, and prefetch plugins. -- 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]
