RFR: 8144100: Incorrect case-sensitive equality in com.sun.net.httpserver.BasicAuthenticator [v2]

Nizar Benalla duke at openjdk.org
Thu May 9 12:49:09 UTC 2024


On Thu, 9 May 2024 12:15:03 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Nizar Benalla has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Declare `ServerAuthenticator.invoked` as volatile
>
> test/jdk/com/sun/net/httpserver/BasicAuthToken.java line 100:
> 
>> 98: 
>> 99:     static class ServerAuthenticator extends BasicAuthenticator {
>> 100:         private static boolean invoked = false;
> 
> this should be declared volatile since it will be mutated in one thread and read in the other.

Fixed in [5003802](https://github.com/openjdk/jdk/pull/19133/commits/5003802b98c1fb3f19703c7c6c5b3aa2ce2bcbc6). I'll run the tests again.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19133#discussion_r1595400259


More information about the net-dev mailing list