search

SpiderOak / SpiderOakMobileClient

SpiderOak HTML5 / Javascript / CSS mobile client application, rebooted
Apache License 2.0
102 stars 27 forks source link

Increase cordova-HTTP network timeouts from 1 minute to 5 #713

Closed kenmanheimer closed 8 years ago

kenmanheimer commented 9 years ago

It looks like the switchover to cordova-HTTP (for cert pinning, etc) reduced the network timeout from our jacked-up javascript XHR setting of five minutes to a default value of 1 minute (here's relevant code for the IOS platform). The linked comment suggests how it can be modified for IOS, Tommy found a pending pull request in the cordova-HTTP original fork, wymsee/cordova-HTTP#40, which would provide for setting the timeout on the Android side (or maybe simpler, convey how we could make that change in a more hard-coded way).

devgeeks commented 9 years ago

I think our old JS timeout was 3 minutes

devgeeks commented 9 years ago

Apparently it was 5.

kenmanheimer commented 9 years ago

@devgeeks, is there any reason not to just go with 5 minutes timeout, and to not include it in the next release? Or 3 minutes, if that seems better - but increase it from what it is now, since that seems likely to be causing ShareRoom timeouts incompatible with the web site?

devgeeks commented 9 years ago

I agree, I just feel like this release is taking a long time. I will push it up to 5 (though I can't imagine ANYONE sitting there with their phone in their hands waiting 5 minutes for something to load).