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

Xuelei Fan xuelei.fan at oracle.com
Fri Jul 19 17:03:06 UTC 2019


The system property parsing behavior should be consistent between JDK 13 
and 14.

Xuelei

On 7/19/2019 9:59 AM, Anthony Scarpino wrote:
> Boolean.parseBoolean will only return true on "true", any other entries 
> will return false.   Since in this case true is the default, I would 
> rather any other entry than "false" to revert to the default.  Therefore 
> going back to the old way is better.
> 
> Tony
> 
> On 7/19/19 1:22 AM, Jaikiran Pai wrote:
>> 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
> 


More information about the security-dev mailing list