RFR: JDK-8268133 : Update java/net/Authenticator tests to eliminate dependency on sun.net.www.MessageHeader and some other internal APIs

Daniel Fuchs dfuchs at openjdk.java.net
Wed Jun 2 18:04:31 UTC 2021


On Wed, 2 Jun 2021 17:22:01 GMT, Mahendra Chhipa <github.com+34924738+mahendrachhipa at openjdk.org> wrote:

> …ency on sun.net.www.MessageHeader and

test/jdk/java/net/Authenticator/B4722333.java line 79:

> 77:                         req.getResponseHeaders().set("Connection", "close");
> 78:                         req.getResponseHeaders().add("WWW-Authenticate", "Basic realm=\"foobar\"");
> 79:                         req.getResponseHeaders().add("WWW-Authenticate", "Foo realm=\"bar\"");

I believe this will change what the test is testing. I suggest replacing these two lines with:


req.getResponseHeaders().add("WWW-Authenticate", "Basic realm="foobar" Foo realm="bar"");


as in the original version of the test.

test/jdk/java/net/Authenticator/B4722333.java line 84:

> 82:                     case 4:
> 83:                         req.getResponseHeaders().set("Connection", "close");
> 84:                         req.getResponseHeaders().set("WWW-Authenticate", "Digest realm=\"biz\" domain=/foo nonce=\"thisisnonce \"");

I'm not very comfortable with changing the logic of the test. Can we keep the strings identical please?

test/jdk/java/net/Authenticator/B4722333.java line 91:

> 89:                         req.getResponseHeaders().set("Connection", "close");
> 90:                         req.getResponseHeaders().set("WWW-Authenticate", "Digest realm=\"bizbar\" domain=/biz nonce=\"hereisanonce\"");
> 91:                         req.getResponseHeaders().add("WWW-Authenticate", "Basic realm=\"foobar\" Foo realm=\"bar\"");

Same remark here. Changing one WWW-Authenticate line into two WWW-Authenticate lines may not trigger the same code path in the client.

test/jdk/java/net/Authenticator/B4722333.java line 98:

> 96:                         req.getResponseHeaders().set("WWW-Authenticate", "Foo p1=1 p2=2 p3=3 p4=4 p5=5 p6=6 p7=7 p8=8 p9=10");
> 97:                         req.getResponseHeaders().add("WWW-Authenticate", "Digest realm=foobiz domain=/foobiz nonce=newnonce");
> 98:                         req.getResponseHeaders().add("WWW-Authenticate", "Basic realm=bizbar");

Here again. Please keep the strings identical.

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

PR: https://git.openjdk.java.net/jdk/pull/4317


More information about the net-dev mailing list