RFR: 8326381: com.sun.net.httpserver.HttpsParameters and SSLStreams incorrectly handle needClientAuth and wantClientAuth [v3]

Jaikiran Pai jpai at openjdk.org
Thu Feb 22 15:31:56 UTC 2024


On Wed, 21 Feb 2024 08:31:43 GMT, John Jiang <jjiang at openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - introduce a test to verify the server honours needClientAuth and wantClientAuth when set through HttpsParameters
>>  - deprecate the SSL parameters related methods on HttpsParameter
>
> test/jdk/com/sun/net/httpserver/HttpsParametersClientAuthTest.java line 49:
> 
>> 47:      */
>> 48:     @Test
>> 49:     public void testClientAuth() throws Exception {
> 
> Just a suggestion.
> 
> Now that this test uses JUnit, why doesn't it define multiple test methods for the different HttpsParameters instances?
> A single test method just focus on only one HttpsParameters instance (or test case).
> 
> If the checking on a HttpsParameters instance (or test case) fails, the remaining test cases still can be executed.

Hello John, I've updated the PR to split these into separate test methods.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17940#discussion_r1499432033


More information about the net-dev mailing list