RFR: 8359477: com/sun/net/httpserver/Test12.java appears to have a temp file race [v3]
Jaikiran Pai
jpai at openjdk.org
Mon Jun 16 11:36:34 UTC 2025
On Mon, 16 Jun 2025 11:27:54 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
> Each new request will be submitted to the executor by the HttpServer, therefore waiting for the executor to finish should ensure that FileServerHandler::handle has finished, and therefore that the the file has been closed.
That's a good point (both your and Volkan's). I missed this detail when Volkan mentioned it because I focused on the part which noted the server would close the file. I will experiment with this (or a different variant that I had in mind) since it looks like the preference is to let this delete calls stay.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25820#discussion_r2149723378
More information about the net-dev
mailing list