Author: titmuss
Date: Wed Oct 29 10:56:15 2008
New Revision: 3229

URL: http://svn.slimdevices.com?rev=3229&root=Jive&view=rev
Log:
Bug: 8159
Description:
Fix another possible cause of the BIOD. If the http headers were not received 
before a 70 second timeout then the connection to the 
server would be lost. This can be tested by using Ctrl-Z on the server for 10 
minutes, then fg.

The fix means that the error is correctly reported. I have also reducted the 
timeout to 10 seconds, as SC should send the headers out 
in a reasonable time.


Modified:
    7.3/trunk/squeezeplay/src/squeezeplay/share/jive/net/SocketHttp.lua

Modified: 7.3/trunk/squeezeplay/src/squeezeplay/share/jive/net/SocketHttp.lua
URL: 
http://svn.slimdevices.com/7.3/trunk/squeezeplay/src/squeezeplay/share/jive/net/SocketHttp.lua?rev=3229&root=Jive&r1=3228&r2=3229&view=diff
==============================================================================
--- 7.3/trunk/squeezeplay/src/squeezeplay/share/jive/net/SocketHttp.lua 
(original)
+++ 7.3/trunk/squeezeplay/src/squeezeplay/share/jive/net/SocketHttp.lua Wed Oct 
29 10:56:15 2008
@@ -468,9 +468,9 @@
        local pump = function (NetworkThreadErr)
                log:debug(self, ":t_rcvHeaders.pump()")
                if NetworkThreadErr then
-                       log:error(self, ":t_rcvHeaders.pump:", err)
+                       log:error(self, ":t_rcvHeaders.pump:", NetworkThreadErr)
                        --self:t_removeRead()
-                       self:close(err)
+                       self:close(NetworkThreadErr)
                        return
                end
 
@@ -520,7 +520,7 @@
                end
        end
        
-       self:t_addRead(pump, SOCKET_TIMEOUT)
+       self:t_addRead(pump, SOCKET_CONNECT_TIMEOUT)
 end
 
 

_______________________________________________
Jive-checkins mailing list
[email protected]
http://lists.slimdevices.com/cgi-bin/mailman/listinfo/jive-checkins

Reply via email to