Print

Print


  Branch: refs/heads/stable-4.8.x
  Home:   https://github.com/xrootd/xrootd
  Commit: 0037189b5d3993c4daf63ab563b965b9f475ca34
      https://github.com/xrootd/xrootd/commit/0037189b5d3993c4daf63ab563b965b9f475ca34
  Author: Brian Bockelman <[log in to unmask]>
  Date:   2018-09-12 (Wed, 12 Sep 2018)

  Changed paths:
    M src/XrdHttp/XrdHttpProtocol.cc

  Log Message:
  -----------
  Do not increment reqstate when headers are incomplete.

If the server got incomplete headers (I was able to reliably trigger
this with connection reuse), the reqstate variable was still incremented.

This resulted in the request state becoming inconsistent - notably,
HEAD requests believed they always included a checksum.  There are
likely other strange XrdHttp behaviors that could have been triggered
due to the same bug.

(cherry picked from commit 88dad6ab1a1062749ba5e9ca5e0e9b98acbcddab)


  Commit: 60f9600e058ae722a52dba96443215a3b5db653e
      https://github.com/xrootd/xrootd/commit/60f9600e058ae722a52dba96443215a3b5db653e
  Author: simonmichal <[log in to unmask]>
  Date:   2018-10-02 (Tue, 02 Oct 2018)

  Changed paths:
    M src/XrdHttp/XrdHttpProtocol.cc

  Log Message:
  -----------
  Merge pull request #825 from bbockelm/fix_partial_headers

Do not increment reqstate when headers are incomplete.


Compare: https://github.com/xrootd/xrootd/compare/9000fa28227d...60f9600e058a
      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.

########################################################################
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