<div dir="ltr"><div dir="ltr">+1<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 17, 2019 at 3:29 PM Daniel Fuchs <<a href="mailto:daniel.fuchs@oracle.com">daniel.fuchs@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Arthur,<br>
<br>
On 17/05/2019 00:16, Arthur Eubanks wrote:<br>
> bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8224081" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8224081</a><br>
> webrev: <a href="http://cr.openjdk.java.net/~aeubanks/8224081/webrev.00/index.html" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~aeubanks/8224081/webrev.00/index.html</a><br>
> <br>
> Tests that try to use SOCKS v4 will fail in an IPv6 only environment <br>
> since SOCKS v4 does not support IPv6. SOCKS v5 does support IPv6.<br>
<br>
Your changes to java/net/Socks/SocksProxyVersion.java look good to me.<br>
<br>
However I'd like your changes to<br>
test/jdk/sun/security/x509/URICertStore/SocksProxy.java<br>
to be reviewed by the security-dev team which I have added<br>
in cc:, since I believe this falls into their area.<br>
<br>
best regards,<br>
<br>
-- daniel<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">Thanks,<br>Vyom</div></div>