bugGNU Wget - Bugs: bug #39780, Wget omits Host header for CONNECT...

 
 

bug #39780: Wget omits Host header for CONNECT (RFC 2616)

Submitter:  Noël Köthe <nok>
Submitted:  Tue 13 Aug 2013 02:09:26 PM UTC
   
 
Category:  Protocol Issue Severity:  3 - Normal
Priority:  5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  trunk Operating System:  GNU/Linux
Reproducibility:  Every Time Fixed Release:  None
Planned Release:  1.15 Regression:  None
Work Required:  None Patch Included:  Yes
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 02 Nov 2013 10:32:14 AM UTC, comment #1: 

the patch is applied upstream now.

Giuseppe Scrivano <gscrivano>
Group administrator
Tue 13 Aug 2013 02:09:26 PM UTC, original submission:  

Hello,

reported to the Debian bug tracker http://bugs.debian.org/699337 and discussed on the wget mailinglist https://lists.gnu.org/archive/html/bug-wget/2013-01/msg00019.html
But the patch from Pavel wasn't applied at that time.

Maybe it could be added.

Inline and attached:

--- a/src/http.c 2011-09-07 12:58:01.000000000 +0200
+++ a/src/http.c 2013-01-30 13:32:42.000000000 +0100
@@ -1847,10 +1847,9 @@
                  the regular request below.  */
               proxyauth = NULL;
             }
-          /* Examples in rfc2817 use the Host header in CONNECT
-             requests.  I don't see how that gains anything, given
-             that the contents of Host would be exactly the same as
-             the contents of CONNECT.  */
+          request_set_header (connreq, "Host",
+                              aprintf ("%s:%d", u->host, u->port),
+                              rel_value);
 
           write_error = request_send (connreq, sock);
           request_free (connreq);

Noël Köthe <nok>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #28833:  connect-host-header added by nok (722B - application/octet-stream - connect-host-header.patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gscrivano (Posted a comment)
  • -email is unavailable- added by nok (Submitted the item)
  • -email is unavailable- added by nok
  •  

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-11-02 gscrivano StatusNone Fixed
        Open/ClosedOpen Closed
        Planned ReleaseNone 1.15
    2013-08-13 nok Attached File- Added connect-host-header, #28833
        Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code