Fwd: kio: slave process dies when using https

[Michael Cardenas]

We're going with your patch, unless qa finds a bug problem with it, but
we also turned the keep-alive header on for keep alive connections. 

I think there's a bug with the fact that the keep-alive header is #if
0'd out, but the keep alive logic is still there, preventing a new
connection from being opened. We tried just turning on the keep alive
header, but it didn't fix the "process died" error. 

thanks for your help

michael


On Fri, May 31, 2002 at 12:35:49AM -0400, Dawit A. wrote:
> Hi Michael,
> 
> Truely sorry about this. Please apply this after applying the patch
> I sent in the second email.  The first part of the patch creeped was
> not supposed to be there!  Basically the fix is to add && !m_bIsSSL in 
> HTTPProtocol::httpClose where we check to see if the connection 
> should be persistent.
> 
> Again my appologies....
> 
> Regards,
> Dawit A.



-- 
michael cardenas
lead windows compatibility engineer
lindows.com
-
hyperpoem.net
-
"Be the change you want to see in the world"
-Mahatma Gandhi
-------------- next part --------------
Index: http.cc
===================================================================
RCS file: /home/cvs/os/kde3/kdelibs/kioslave/http/http.cc,v
retrieving revision 1.1
retrieving revision 1.2
diff -r1.1 -r1.2
1543c1543
<   } while (( ret == -1) && ((errno == EAGAIN) || (errno == EINTR)));
---
>   } while (( ret == -1) && (m_bIsSSL || (errno == EAGAIN) || (errno == EINTR)));
1792c1792
<   }
---
>     }
1937c1937,1939
< #if 0
---
> 
>     //mbc - put the keep-alive header back in!
>     //#if 0
1943c1945
< #endif
---
>     //#endif
2119a2122
> 
3166c3169,3170
<   if (m_bKeepAlive && m_request.method == HTTP_GET)
---
> 
>   if (m_bKeepAlive && !m_bIsSSL && m_request.method == HTTP_GET)
3170c3174,3176
<   }
---
>     }
> 
>   //mbc - turn off persistent connections