RFR: 8358048: java/net/httpclient/HttpsTunnelAuthTest.java incorrectly calls Thread::stop

Jaikiran Pai jpai at openjdk.org
Fri Jul 25 08:02:48 UTC 2025


Can I please get a review of this test-only change which fixes an accidental use of `Thread.stop()`? This addresses https://bugs.openjdk.org/browse/JDK-8358048.

The test during it's termination attempts to close the `ProxyServer` and invokes `ProxyServer.stop()`. The `ProxyServer` does not have a `stop()` method of its own but since the `ProxyServer` is a `java.lang.Thread`, calling `stop()` on the `ProxyServer` ends up calling the `Thread.stop()`. The use of `stop()` in this test is accidental and the test should have used `ProxyServer.close()`.

The commit in this PR fixes the issue by removing the use of `ProxyServer.stop()`. The test continues to pass with this change.

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

Commit messages:
 - 8358048: java/net/httpclient/HttpsTunnelAuthTest.java incorrectly calls Thread::stop

Changes: https://git.openjdk.org/jdk/pull/26473/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=26473&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8358048
  Stats: 6 lines in 1 file changed: 1 ins; 1 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/26473.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26473/head:pull/26473

PR: https://git.openjdk.org/jdk/pull/26473


More information about the net-dev mailing list