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

Nizar Benalla duke at openjdk.org
Thu May 9 13:12:53 UTC 2024


On Wed, 8 May 2024 11:44:42 GMT, Jaikiran Pai <jpai 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 28:
> 
>> 26:  * @bug 8144100
>> 27:  * @run main/othervm BasicAuthToken
>> 28:  * @summary checking token sent by client should be done in case-insensitive manner
> 
> As a guideline, jtreg recommended order of test definition tags is here https://openjdk.org/jtreg/tag-spec.html#ORDER. As per that the `@summary` should follow the `@bug` before the `@run`.

Done

> test/jdk/com/sun/net/httpserver/BasicAuthToken.java line 97:
> 
>> 95: 
>> 96:     static class ServerAuthenticator extends BasicAuthenticator {
>> 97:         ServerAuthenticator(String realm) {
> 
> Since this test relies on the authenticator being invoked, just as an additional check, you might want to have a boolean field in this class (named something like `invoked`) which will be set to `true` only when the `checkCredentials` gets called. Then after checking the response code in the test, you can additionally assert that this field's value is `true`, just to be certain that this authenticator was indeed invoked.

Fixed, thanks

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

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


More information about the net-dev mailing list