ChangeVision / proxy-vole

Automatically exported from code.google.com/p/proxy-vole
0 stars 0 forks source link

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

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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