RFR: 8161016: Strange behavior of URLConnection with proxy

Aleksey Shipilev aleksey.shipilev at gmail.com
Fri Aug 12 20:46:52 UTC 2016


On 08/12/2016 10:56 PM, Ramanand Patil wrote:
> Here is the updated Webrev: http://cr.openjdk.java.net/~rpatil/8161016/webrev.01/

I think my version (below) is a little more straightforward, but no
pressure. Your version is okay too, and does the minimal change. Thumbs up.

Thanks,
-Aleksey

> } catch (IOException ioex) {
>   if (p == Proxy.NO_PROXY) {
>     throw ioex;
>   }
>   sel.connectFailed(uri, p.address(), ioex);
>   if (it.hasNext()) {
>      continue;   // try the next Proxy
>   } else {
>      throw ioex; // that was the last Proxy, time to fail
>   }
> }
> 
> Thanks,
> -Aleksey
> 




More information about the core-libs-dev mailing list