RFR: JDK-8212035 : merge jdk.test.lib.util.SimpleHttpServer with jaxp.library.SimpleHttpServer [v5]

Daniel Fuchs dfuchs at openjdk.java.net
Thu Dec 17 17:15:03 UTC 2020


On Thu, 17 Dec 2020 16:24:10 GMT, Mahendra Chhipa <github.com+34924738+mahendrachhipa at openjdk.org> wrote:

>> jaxp.library.SimpleHttpServer
>> https://bugs.openjdk.java.net/browse/JDK-8212035
>
> Mahendra Chhipa has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
> 
>  - Implemented the Review comments.
>  - Implemented the review comments
>  - Merge branch 'JDK-8212035' of https://github.com/mahendrachhipa/jdk into JDK-8212035
>  - Implemnted the review comments.
>  - Implemnted the review comments.
>  - Implemented the review comments.
>  - JDK-8212035 merge jdk.test.lib.util.SimpleHttpServer with
>    jaxp.library.SimpleHttpServer
>    https://bugs.openjdk.java.net/browse/JDK-8212035

test/lib/jdk/test/lib/net/SimpleHttpServer.java line 104:

> 102:                     try {
> 103:                         uri = URI.create(rootUri.getRawPath() + path).normalize();
> 104:                         fPath = Path.of(uri.getRawPath());

This is wrong. It should be `fpath = Path.of(uri);`. See https://docs.oracle.com/en/java/javase/15/docs/api/java.base/java/nio/file/Path.html#of(java.net.URI)

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

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


More information about the net-dev mailing list