This commit updates the handling of HTTP headers to conform to HTTP standards and improve support for chunked transfer encoding and status trailers.
Previously, the server incorrectly expected to receive the "Transfer-Encoding: chunked" header in HTTP requests. However, it has been realized that the "Transfer-Encoding" header is reserved for HTTP responses and should not be anticipated in an HTTP request.
We've now corrected this behavior by removing the check for the "Transfer-Encoding" header in the incoming request. Instead, we now set the internal variable `m_transfer_encoding_chunked` when the client includes the custom header "X-Transfer-Status: true". This change aligns with the original intention, indicating that the client is prepared to receive a response using chunked transfer encoding and a status trailer.
The `m_trailer_headers` variable is still set when the client includes the "TE: trailers" header, indicating its acceptance of trailer fields in the response.
You can view, comment on, or merge this pull request online at:
https://github.com/xrootd/xrootd/pull/2009
-- Commit Summary --
* Update HTTP Header Handling for Chunked Encoding and Status Trailer
-- File Changes --
M src/XrdHttp/XrdHttpReq.cc (3)
-- Patch Links --
https://github.com/xrootd/xrootd/pull/2009.patch
https://github.com/xrootd/xrootd/pull/2009.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/2009
You are receiving this because you are subscribed to this thread.
Message ID: <[log in to unmask]>
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
