On Fri, 22 Nov 2024 19:47:53 GMT, Phil Race <p...@openjdk.org> wrote:
>> Will changes **sun/net/util/URLUtil.java** be handled in >> [JDK-8344180](https://bugs.openjdk.org/browse/JDK-8344180)? Or do we do it >> here? > >> so the jdk/src/java.base/share/classes/sun/net/util/URLUtil.java part can be >> reverted > > true > >> will changes to java.base files - URLUtil.java be handled in >> [JDK-8344180](https://bugs.openjdk.org/browse/JDK-8344180)? Or do we do it >> here? > > If really no other code is using, it then yes, you could do it here but you > should ask @dfuch about that since I don't know what is in his mind .. > > In the bug report he just wrote "This will allow to remove the method from > sun.net.util.URLUtil." > > Which could be interpreted either way. > Will changes sun/net/util/URLUtil.java be handled in > [JDK-8344180](https://bugs.openjdk.org/browse/JDK-8344180)? It has only two subtasks with mentions of `URLUtil`: - [JDK-8344221](https://bugs.openjdk.org/browse/JDK-8344221) it is already resolved - this issue So I guess the `getConnectPermission` removal can be part of this PR. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22330#discussion_r1854581718
