RFR[14] Re-enable Stateless Resumption On by default for merge to mainline

Jaikiran Pai jai.forums2013 at gmail.com
Fri Jul 19 08:22:48 UTC 2019


Actually, looking at this current code in JDK 14 (default branch), it
uses the "compareToIgnoreCase" version. So I guess, you did this change
to match with what's already there. So please ignore my previous comment.

-Jaikiran

On 19/07/19 12:18 PM, Jaikiran Pai wrote:
>
> Hello Tony,
>
> I'm not a reviewer, but just checking - is this following change
> intentional:
>
> -            statelessSession = Boolean.parseBoolean(st);
> +            if (st.compareToIgnoreCase("false") == 0) {
> +                statelessSession = false;
> +            }
>
> The Boolean.parseBoolean, that's in JDK 13, should work here too.
>
> -Jaikiran
> On 18/07/19 11:19 PM, Anthony Scarpino wrote:
>> This is a changeset for preventing Stateless Resumption from being
>> turned off when JDK-8227551 is merged from 13 into the mainline repo.
>> This is the way the gatekeepers asked me to do it as oppose to
>> withholding 8227551 from the mainline merge.
>>
>> http://cr.openjdk.java.net/~ascarpino/8228396/webrev/
>>
>> Thanks
>>
>> Tony
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/security-dev/attachments/20190719/451c3d68/attachment-0001.html>


More information about the security-dev mailing list