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

Weijun Wang weijun at openjdk.org
Thu Aug 4 01:43:53 UTC 2022


On Tue, 2 Aug 2022 05:54:13 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 change the test case

I meant you can add **both** `UnixLoginModule` and `NTLoginModule` as OPTIONAL. On Windows, `UnixLoginModule` would fail, and on *nix, `NTLoginModule` will fail. Then you can observe the newly added debug info.

BTW, have you reverted your change on `LoginContext.java`. I remembered we agreed that this bug is all about debug info instead of updating the exception thrown.

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

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



More information about the security-dev mailing list