RFR: 8284105: Update security libraries to use sealed classes [v2]

Weijun Wang weijun at openjdk.java.net
Fri Apr 8 21:35:52 UTC 2022


On Fri, 8 Apr 2022 20:07:32 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Please review these changes to update the security libraries to use sealed classes. See JEP 409 for more details on sealed classes.
>> 
>> No CSR is required as all the changes are to internal classes. A few classes that did not have subclasses were simply marked final instead of sealed.
>
> Sean Mullan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Make some classes package-private instead of sealed.

Only one comment. Others look fine.

src/java.security.jgss/share/classes/sun/security/krb5/KrbTgsRep.java line 43:

> 41:  * Kerberos client.
> 42:  */
> 43: final class KrbTgsRep extends KrbKdcRep {

Make `KrbAsRep` final also to be symmetric.

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

Marked as reviewed by weijun (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8165



More information about the security-dev mailing list