RFR: JDK-8344303 : Remove usage of URLUtil.getConnectPermission from sun.awt.SunToolkit and sun.awt.image.URLImageSource [v2]
Daniel Fuchs
dfuchs at openjdk.org
Fri Nov 22 20:14:16 UTC 2024
On Fri, 22 Nov 2024 20:00:14 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:
>>> 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.
Hi @azvegint, if you have no objection to include the deletion of `URLUtil.getConnectPermission()` in this PR, then please do.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22330#discussion_r1854596005
More information about the client-libs-dev
mailing list