This fix looks fine to me.
Thanks.
> On 31 Mar 2017, at 16:34, Sean Mullan wrote:
>
> New webrev uploaded to address your comments:
> http://cr.openjdk.java.net/~mullan/webrevs/8175029/webrev.01/
>
> --Sean
>
> On 3/31/17 9:11 AM, Sean Mullan wrote:
>> On 3/31/17 9:08 AM, Weijun Wang wrote:
New webrev uploaded to address your comments:
http://cr.openjdk.java.net/~mullan/webrevs/8175029/webrev.01/
--Sean
On 3/31/17 9:11 AM, Sean Mullan wrote:
On 3/31/17 9:08 AM, Weijun Wang wrote:
Can we just move the code you just added into the public API directly?
Looks like it does not refere
Hi,
both errors have in common that there are about an change of allowed
hostnames.
But as for the second bug there is an big mistake. The underscore is
allowed for SRV and CNAME
but not for A records. For SRV records it is wide used in LDAP records
for MS Domain System (PDC).
But the Subject
On 3/31/17 9:08 AM, Weijun Wang wrote:
Can we just move the code you just added into the public API directly?
Looks like it does not reference any internal classes.
Good point, that should work! I'll try that and send out an updated
webrev later.
--Sean
--Max
On 03/31/2017 08:06 PM, Sean
Can we just move the code you just added into the public API directly?
Looks like it does not reference any internal classes.
--Max
On 03/31/2017 08:06 PM, Sean Mullan wrote:
Please review this fix for a StackOverflowError caused by the default
implementation of verify(PublicKey, Provider).
b
Please review this fix for a StackOverflowError caused by the default
implementation of verify(PublicKey, Provider).
bug: https://bugs.openjdk.java.net/browse/JDK-8175029
webrev: http://cr.openjdk.java.net/~mullan/webrevs/8175029/webrev.00/
Thanks,
Sean
