Feedback on JSR-221 maintenance release 4 (JDBC 4.4)

[Lance Andersen]

As indicated in JEP 486<https://openjdk.org/jeps/486>:

——————
In a future release we will deprecate Permission and related classes such as BasicPermission, PermissionCollection, and Permissions, and also subclasses of Permission outside of the java.security package, such as java.lang.RuntimePermission, java.net.NetPermission, and java.lang.reflect.ReflectPermission.

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

Once the above occurs the same will be done for other classes such as SQLPermission


As a side note, I will be working on a small JDBC MR tentatively targeted for JDK 26 at which point I hopefully will be in position to deprecate this class and other Permission related classes within the JDK


On Dec 12, 2024, at 10:15 AM, Mark Rotteveel <mark at lawinegevaar.nl> wrote:

Looking at the changelog in https://jcp.org/aboutJava/communityprocess/maintenance/jsr221/index4.html, and specifically the javadoc diff from https://jcp.org/aboutJava/communityprocess/maintenance/jsr221/221ChangeLog.html, I think the changed to SQLPermission should be reverted, and instead it should be deprecated (for removal, assuming BasicPermission also gets deprecated for removal?), as I think it serves no purpose when the SecurityManager is disabled/removed.

Without further context, the fact only the second paragraph is retained is confusing IMHO, and for example the see "(see below)" no longer applies with the removal of the table of permission names.

I think the original description should be preserved, and together with deprecation and an explanation why it is no longer used is better than the current change.

Mark
--
Mark Rotteveel


[oracle_sig_logo.gif]






Lance Andersen | Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/jdbc-spec-discuss/attachments/20241212/02257f16/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: oracle_sig_logo.gif
Type: image/gif
Size: 658 bytes
Desc: oracle_sig_logo.gif
URL: <https://mail.openjdk.org/pipermail/jdbc-spec-discuss/attachments/20241212/02257f16/oracle_sig_logo.gif>