DefaultProxySelector socks override

[Chris Hegarty]

Thank you Kurchi for taking this in, and running/updating the test.

Do you think that anyone might be surprised by this change, like our 
test was? Wanting to proxy connections to the localhost seems more like 
testing environments rather than deployment.

-Chris.

On 04/03/2013 01:13 AM, Kurchi Hazra wrote:
> Hi Christos/Chris,
>
>     Here is a webrev for this change:
> http://cr.openjdk.java.net/~khazra/5001942/webrev.00/
>
> Thanks,
> - Kurchi
>
>
>
> On 3/27/2013 10:49 AM, christos at zoulas.com wrote:
>> On Mar 27,  5:30pm, chris.hegarty at oracle.com (Chris Hegarty) wrote:
>> -- Subject: Re: DefaultProxySelector socks override
>>
>> | On 03/27/2013 05:22 PM, christos at zoulas.com wrote:
>> | > ....
>> | > Sure, I just requested a subscription to net-dev so I might not
>> see the
>> | > first few messages. To clarify:
>> | >
>> | >     1. I will add socks.proxyHost and socks.proxyPort for consistency
>> | >        with the other protocols, leaving as is socksProxyHost and
>> | >        socksProxyPort for compatibility.
>> | >     2. I will add socks.nonProxyHosts and not socksNonProxyHosts.
>> | >
>> | > Is that what you had in mind?
>> |
>> | Re-checking the code I take back my previous comment. We already have
>> |
>> |     socksProxyHost, socksProxyPort, socksProxyVersion
>> |
>> | so your original proposal of 'socksNonProxyHosts' is probably best, and
>> | consistent with existing properties.
>>
>> I concur. Nothing for me to do :-)
>>
>> Best,
>>
>> christos
>