RFR: 8215916: The failure reason of an optional JAAS LoginModule is not logged [v22]

Weijun Wang weijun at openjdk.org
Mon Aug 22 19:49:59 UTC 2022


On Mon, 22 Aug 2022 18:39:14 GMT, Jayashree Huttanagoudar <duke at openjdk.org> wrote:

>> Could you please review the changes?
>> This patch is to address : https://bugs.openjdk.org/browse/JDK-8215916?jql=labels%20%3D%20starter-bug
>
> Jayashree Huttanagoudar has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address review comment to update copyright

Approved.

Some tiny comments (sorry for more):

1. No need to import `Configuration` class now.
2. No need to import `Paths` class now, or, you can change it to `Path` and simplify its reference on line 52.

No need for more code review for these changes.

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

Marked as reviewed by weijun (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9159


More information about the security-dev mailing list