[PATCH] JDK-8226976 - SessionTimeOutTests uses == operator for String value check
Xuelei Fan
Xuelei.Fan at Oracle.Com
Tue Jul 2 05:33:24 UTC 2019
The copyright date format is similar to “.. 2001, 2019, ..”.
Xuelei
> On Jul 1, 2019, at 10:22 PM, Jaikiran Pai <jai.forums2013 at gmail.com> wrote:
>
>
>> On 02/07/19 7:35 AM, Jaikiran Pai wrote:
>> Thank you Xuelei. Here's the updated webrev
>> http://cr.openjdk.java.net/~jpai/webrev/8226976/01/webrev/.
> - * Copyright (c) 2001, 2017, Oracle and/or its affiliates. All rights reserved.
> + * Copyright (c) 2001, 2017, 2019 Oracle and/or its affiliates. All rights reserved.
> I just checked some other files within the JDK and can't seem to find a copyright with more than 2 years listed. So if you meant that I should change it to "...2001, 2019..." then please do let me know and I'll update it accordingly and send the updated webrev.
>
> -Jaikiran
>
>
>
>> Updated the
>> copyright year and also added you as the reviewer, in the commit log
>> message.
>>
>> -Jaikiran
>>
>> On 01/07/19 10:17 PM, Xuelei Fan wrote:
>>> Hi Jaikiran,
>>>
>>> Would you mind update the copyright date to 2019? Otherwise, looks
>>> good to me.
>>>
>>> Thanks,
>>> Xuelei
>>>
>>> On 6/29/2019 6:03 AM, Jaikiran Pai wrote:
>>>> While investigating a potential regression in JDK 13+, I happened to use
>>>> the test/jdk/javax/net/ssl/SSLSession/SessionTimeOutTests.java. This
>>>> test incorrectly uses the == operator for one of the String checks. I
>>>> have filed a bug[1] with the details and have a patch with the fix
>>>> here[2].
>>>>
>>>> Can I please get a review and a sponsor for this change?
>>>>
>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8226976
>>>>
>>>> [2] http://cr.openjdk.java.net/~jpai/webrev/8226976/0/webrev/
>>>>
>>>> -Jaikiran
>>>>
>>>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20190701/7a8746a8/attachment.htm>
More information about the security-dev
mailing list