Menu ▾ ▴

#1094 An update of case 3491697

closed-fixed
Daniel Stenberg
https (67)
5
2013-06-21
2012-02-24
Anonymous
No

Hi,

About the ipv6 and proxy case.
Thanks for your information.,
I have test with the patch from the case ID: 3482093.
And I found there is still one more place need to be modified.

lib/http_proxy.c
Around line 147 ~ 153

Original:
Curl_add_bufferf(req_buffer,
"CONNECT %s:%hu HTTP/%s\r\n"
....
hostname, ....

New:
Curl_add_bufferf(req_buffer,
"CONNECT %s%s%s:%hu HTTP/%s\r\n"
....
conn->bits.ipv6_ip?"[":"", hostname, conn->bits.ipv6_ip?"]":"", ........

Just as the changing on line 124.

My https query returns 200 after this change.
Please check this, thanks.

Regards,
Blacat

Discussion

  • Daniel Stenberg

    Daniel Stenberg - 2012-02-24

    Thanks for the report, this problem is now fixed in the git repository.

    To try it out, you either checkout/update your git clone: http://curl.haxx.se/source.html

    or you try tomorrow's daily snapshot: http://curl.haxx.se/snapshots/

     

  • Daniel Stenberg

    Daniel Stenberg - 2012-02-24
    • status: open --> closed-fixed