8348986: Improve coverage of enhanced exception messages by alexeybakhtin · Pull Request #3989 · openjdk/jdk17u-dev

alexeybakhtin · 2025-09-23T19:49:19Z

I'd like to backport JDK-8348986 to control sensitive information in Exception messages.

Backport from openjdk/jdk21u-dev#2132 is not clean. The following changes were applied manually:

src/java.base/share/classes/sun/nio/ch/DatagramSocketAdaptor.java
- copyright year
src/java.base/share/classes/sun/nio/ch/NioSocketImpl.java
- Throwing IOException from the connect() method is updated manually because of a context difference
src/java.base/share/classes/module-info.java
- exports sun.security.util updated manually
src/java.base/share/classes/java/net/Inet4AddressImpl.java
- changes discarded. JDK17 does not have JDK-8244202 and related changes : lookupAllHostAddr(String hostname, LookupPolicy lookupPolicy)
src/java.base/share/classes/java/net/InetAddress.java
- throwing UnknownHostException from NameServiceAddresses::get is updated manually because of context differences
- import section
- no CachecdLookup and ValidCachedLookup classes in JDK17. changes discarded
src/java.base/share/classes/java/net/Proxy.java
- copyright year
src/java.base/share/classes/java/net/SocketPermission.java
- copyright year
src/java.base/share/classes/java/net/SocksSocketImpl.java
- copyright year
src/java.base/share/classes/java/net/URI.java
- import section
src/java.base/share/classes/java/net/URL.java
- copyright year
src/java.base/share/classes/java/net/URLStreamHandler.java
- copyright year
src/java.base/share/classes/sun/net/www/protocol/jar/Handler.java
- copyright year
src/java.base/windows/native/libnet/Inet4AddressImpl.c
- copyright year
src/java.base/windows/native/libnet/Inet6AddressImpl.c
- copyright year
src/java.base/share/native/libnet/net_util.h
- copyright year
- getEnhancedExceptionsAllowed() declaration added manually because of context difference
src/java.naming/share/classes/com/sun/jndi/toolkit/url/Uri.java
- parseCompat method is updated manually because of context differences
src/java.base/windows/classes/sun/nio/ch/WindowsAsynchronousSocketChannelImpl.java
- copyright year
src/java.base/windows/classes/sun/nio/fs/WindowsUserPrincipals.java
- copyright year
- lookup method is updated manually because of context differences
src/jdk.httpserver/share/classes/sun/net/httpserver/simpleserver/JWebServer.java
- no such class in JDK17

Modified and related JTREG tests are passed

Progress

Change must be properly reviewed (1 review required, with at least 1 Reviewer)
JDK-8348986 needs maintainer approval
Change must not contain extraneous whitespace
Commit message must refer to an issue
Change requires CSR request JDK-8366459 to be approved

Issues

JDK-8348986: Improve coverage of enhanced exception messages (Enhancement - P3)
JDK-8366459: Improve coverage of enhanced exception messages (CSR)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/3989/head:pull/3989
$ git checkout pull/3989

Update a local copy of the PR:
$ git checkout pull/3989
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/3989/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 3989

View PR using the GUI difftool:
$ git pr show -t 3989

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/3989.diff

Using Webrev

Link to Webrev Comment

bridgekeeper · 2025-09-23T19:50:55Z

👋 Welcome back abakhtin! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

openjdk · 2025-09-23T19:52:03Z

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

openjdk · 2025-09-23T19:52:16Z

This backport pull request has now been updated with issue from the original commit.

mlbridge · 2025-09-23T19:55:53Z

Webrevs

01: Full - Incremental (55181504)
00: Full (5460df7e)

sercher · 2025-09-29T11:25:36Z

Hi @alexeybakhtin
I noticed that sun/net/util/SocketExceptions.java changed to an empty file. Is it as expected?

sercher · 2025-09-29T12:35:17Z

src/java.base/share/classes/sun/net/www/protocol/jmod/Handler.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.


Only the copyright header changed

Thank you. Fixed

sercher · 2025-09-29T12:36:06Z

src/java.base/share/classes/sun/net/util/SocketExceptions.java

@@ -1,104 +0,0 @@
-/*


SocketExceptions becomes an empty file

Thank you. Removed

sercher · 2025-09-29T13:11:31Z

src/java.base/share/classes/module-info.java

+        java.naming,
+        java.rmi,


sercher · 2025-09-29T13:26:54Z

src/java.base/share/classes/module-info.java

+        java.naming,
+        java.rmi,
+        java.net.http,
        java.xml.crypto,
        jdk.crypto.ec,
        jdk.crypto.cryptoki,
+        jdk.httpserver,


This change must be for jdk.internal.util

Thank you. Moved

sercher · 2025-09-29T13:52:22Z

src/java.base/share/classes/sun/nio/ch/SocketChannelImpl.java

            // connect failed, close the channel
            close();
-            throw SocketExceptions.of(ioe, sa);
+            throw Exceptions.ioException(ioe, sa);


A similar change is needed also in AbstractPlainSocketImpl.java

Yes. Thank you. Fixed.

…tp module

bridgekeeper · 2025-10-28T16:58:45Z

@alexeybakhtin This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!

bridgekeeper · 2025-11-26T07:18:55Z

@alexeybakhtin This pull request has been inactive for more than 8 weeks and will now be automatically closed. If you would like to continue working on this pull request in the future, feel free to reopen it! This can be done using the /open pull request command.

alexeybakhtin · 2026-02-04T00:09:55Z

/open

openjdk · 2026-02-04T00:10:06Z

@alexeybakhtin This pull request is now open

Backport b6f827ef054959662190e21ce63fc3d3c45b92f3

5460df7

openjdk bot changed the title ~~Backport b6f827ef054959662190e21ce63fc3d3c45b92f3~~ 8348986: Improve coverage of enhanced exception messages Sep 23, 2025

openjdk bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels Sep 23, 2025

sercher reviewed Sep 29, 2025

View reviewed changes

src/java.base/share/classes/module-info.java Outdated

Comment on lines 339 to 340

java.naming,

java.rmi,

Copy link

Contributor

sercher Sep 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicate

sercher reviewed Sep 29, 2025

View reviewed changes

Add accessClassInPackage.jdk.internal.util permission for java.net.ht…

33997ab

…tp module

alexeybakhtin marked this pull request as draft September 29, 2025 23:05

openjdk bot removed the rfr Pull request is ready for review label Sep 29, 2025

Fix review findings

5518150

alexeybakhtin marked this pull request as ready for review September 30, 2025 13:58

openjdk bot added the rfr Pull request is ready for review label Sep 30, 2025

bridgekeeper bot closed this Nov 26, 2025

openjdk bot reopened this Feb 4, 2026

Conversation

alexeybakhtin commented Sep 23, 2025 • edited by openjdk bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Progress

Issues

Reviewing

Uh oh!

bridgekeeper bot commented Sep 23, 2025

Uh oh!

openjdk bot commented Sep 23, 2025

Uh oh!

openjdk bot commented Sep 23, 2025

Uh oh!

mlbridge bot commented Sep 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Webrevs

Uh oh!

sercher commented Sep 29, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bridgekeeper bot commented Oct 28, 2025

Uh oh!

bridgekeeper bot commented Nov 26, 2025

Uh oh!

alexeybakhtin commented Feb 4, 2026

Uh oh!

openjdk bot commented Feb 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

alexeybakhtin commented Sep 23, 2025 •

edited by openjdk bot

Loading

mlbridge bot commented Sep 23, 2025 •

edited

Loading