[Sugar-devel] Patch review request for ticket #2290

[Marco Pesenti Gritti]

On Sun, Sep 5, 2010 at 3:44 PM, Dipankar Patro <dipankar at seeta.in> wrote:
> Hi,
>
> In reference to ticket #2290 (http://bugs.sugarlabs.org/ticket/2290)
>
> I have uploaded a patch (have attached it too).
>
> Problem: in the register procedure the time out of connection was not
> implemented, for an unavailable server.
>
> Solution in Patch:
> - Have implemented a TimedOut connection (TimedOutServerProxy( ) function)
> which encounters the problem of unavailable servers.
> - The connection timeout can be changed in seconds through the variable :
> 'timeout'.
> The default setting is at : timeout=socket._GLOBAL_DEFAULT_TIMEOUT
> Change it to : timeout=10, to find Sugar does not freeze anymore.

How long is the default timeout? Are we freezing because we are doing
the xmlrpc requests synchronously?

Marco