RFR: 8367114: Update jdk.test.lib.net.SimpleHttpServer to use SimpleFileServer [v2]
Mahendra Chhipa
mchhipa at openjdk.org
Fri Oct 3 17:40:47 UTC 2025
On Fri, 3 Oct 2025 05:33:41 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Hello Mahendra, it's good that we are moving to use the `com.sun.net.httpserver.SimpleFileServer` in this test library class. However, it looks like before this change, this test library class was using a `Executor` which allowed it to handle multiple requests concurrently and thus serve the files concurrently. With this change, the requests will now be handled serially because the lack of any executor being configured on the internal `HttpServer` instance backing the `SimpleFileServer` instance.
>
> Looking at the references of this `jdk.test.lib.net.SimpleHttpServer` test library class, I see only 3 tests using it. I wonder if we should just delete this class altogether and instead using the the `SimpleFileServer` at the call sites as and how necessary.
Thanks. Now SimpleFileServer is being used in tests and jdk.test.lib.net.SimpleHttpServer is deleted now.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27608#issuecomment-3366616592
More information about the net-dev
mailing list