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