brsanthu / proxy-vole

Automatically exported from code.google.com/p/proxy-vole

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Default timeout is used when fetching PAC script, causes 5 minute delay during app startup if PAC script is not available

GoogleCodeExporter opened this issue · comments

What steps will reproduce the problem?
1. Configure a PAC script location which allows connect, but never returns the 
request, i.e. a connect to the url will hang until a timeout occurs
2. proxy-vole blocks the select() call for 5 minutes, which seems to be the 
default timeout of a HttpURLConnection

What is the expected output? What do you see instead?
* We should use a shorter socket timeout in the HttpURLConnection
* It seems the code integrated into the Android Proxy App at 
https://github.com/madeye/proxydroid/tree/master/src/com/btr/proxy/selector/pac 
has a connection timeout, maybe we simply need to sync these two.

What version of the product are you using? On what operating system?
* 20120727, but 20121203 seems to have the same issue.

Original issue reported on code.google.com by Dominik....@gmail.com on 7 Mar 2013 at 1:28

Hello Dominik,
Yes this makes sense. I'm not really aware which other products integrate 
proxy-vole and therefore it is hard for me to integrate changes that they might 
have done but never reported back to me. 
But in this case it seems I can steal some code from them to set a better 
timeout.

Thanks for reporting this,
- Rossi


Original comment by rosstaus...@googlemail.com on 7 Mar 2013 at 5:16

  • Changed state: Accepted
Hello Dominik,
Yes this makes sense. I'm not really aware which other products integrate 
proxy-vole and therefore it is hard for me to integrate changes that they might 
have done but never reported back to me. 
But in this case it seems I can steal some code from them to set a better 
timeout.

Thanks for reporting this,
- Rossi


Original comment by rosstaus...@googlemail.com on 7 Mar 2013 at 5:16

Hi,
This issue is fixed now. I have set smaller default connection timout of 15 
seconds and a read timeout of 20 seconds.
These can also be set via the system properties. 

com.btr.proxy.url.connectTimeout
and
com.btr.proxy.url.readTimeout

This will be included in the next release that is comming this week.

Have fun,
- Rossi

Original comment by rosstaus...@googlemail.com on 3 Dec 2013 at 12:15

  • Changed state: Fixed