Please review Line 649-->653 in the file attached, thanks. 

     On Monday, February 9, 2015 8:45 PM, Daniel Stenberg <[email protected]> 
wrote:
   

 On Thu, 5 Feb 2015, laser peter via curl-library wrote:

> I mean https proxy there in my env.

Yes, but that is still a HTTP proxy. Without the S! =)

> Could you please review the patch? It verified fixed the issue in my ENV. ++ 
>  /* Bug fix */++  if (rc == 0) {++    sock[0] = conn->sock[FIRSTSOCKET];++   
>  rc = GETSOCK_WRITESOCK(0);++  }+++   /* when we've sent a CONNECT to a 
> proxy, we should rather wait for the+      socket to become readable to be 
> able to get the response headers */+   if(conn->tunnel_state[FIRSTSOCKET] == 
> TUNNEL_CONNECT) {

I'd love to!

Can you please send a full patch[*] that isn't HTML or otherwise scrambled 
(like perhaps attached) when it reaches the list?

[*] = http://curl.haxx.se/dev/contribute.html#How_To_Make_a_Patch_with_git

-- 

  / daniel.haxx.se

   

Attachment: multi.c.patch
Description: Binary data

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to