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