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

Weijun Wang weijun at openjdk.org
Tue Aug 16 14:36:12 UTC 2022


On Tue, 16 Aug 2022 13:49:34 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 for cross-platform in the test case

The original "cross-platform" case is exactly what this bug is about. The overall login succeeds but the reason of the optional login module failure is not logged in the debug output. You need to update the source code of `LoginContext.java` to print out the reason and then confirm it in the test by examining the content of `new String(byes)`.

Until now, you still only have a test change in your commits. This is not a test-only bug.

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

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


More information about the security-dev mailing list