RFR: 8367114: Update jdk.test.lib.net.SimpleHttpServer to use SimpleFileServer

Mahendra Chhipa mchhipa at openjdk.org
Fri Oct 3 17:44:46 UTC 2025


On Fri, 3 Oct 2025 05:41:58 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Update SimpleHttpServer class to use SimpleFileServer.
>
> Interestingly, there's even a `test/jdk/com/sun/net/httpserver/SimpleFileServer.java` class which is neither a test nor a test library class. As far as I can see, it's not being used anywhere. Maybe we should remove that one too. These similarly named classes add to confusion.

> The SimpleHttpServer class that Mahendra is updating is used by multi-relase jars and jaxp catalog tests. It could be prudent to update the class first, check that everything remains stable, and then replace usage in a followup issue. It seems this class is only being used by three tests however - so maybe updating those three tests to use the SimpleFileServer directly won't be too bad. If we do that then make sure we keep the same behaviour = keep the executor as @jaikiran suggested.
> 
> Good find about `test/jdk/com/sun/net/httpserver/SimpleFileServer.java` - let's attempt to remove it - it does not appear to be used anywhere.

Thanks, Now test/jdk/com/sun/net/httpserver/SimpleFileServer.java is removed.

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

PR Comment: https://git.openjdk.org/jdk/pull/27608#issuecomment-3366628509


More information about the net-dev mailing list