> Can I please get a review of this change which proposes to remove the > confusing "getsockopt" text from the exception reported on Windows when > `SocketChannel.finishConnect()` raises an exception due to a failed > `connect()`? > > This addresses https://bugs.openjdk.org/browse/JDK-8376290 and as noted in > that issue, this looks like an oversight when changes were done for > https://bugs.openjdk.org/browse/JDK-8317603. Prior to that, on older versions > the exception reported on Windows was: > > > java.net.ConnectException: Connection refused: no further information > > > With the current proposed changes in this PR, the exception will now be > `java.net.ConnectException: Connection refused` which matches what we report > on *nix platforms. > > A new jtreg test has been added to reproduce the original text and verify the > change. tier1, tier2 and tier3 continue to pass with this change.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: skip the test if the SocketChannel unexpectedly connects to the destination address ------------- Changes: - all: https://git.openjdk.org/jdk/pull/29409/files - new: https://git.openjdk.org/jdk/pull/29409/files/8d391779..2e555591 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=29409&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29409&range=01-02 Stats: 31 lines in 1 file changed: 11 ins; 5 del; 15 mod Patch: https://git.openjdk.org/jdk/pull/29409.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29409/head:pull/29409 PR: https://git.openjdk.org/jdk/pull/29409
