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

Tue Aug 16 14:42:17 UTC 2022

On Tue, 16 Aug 2022 14:32:46 GMT, Weijun Wang <weijun at openjdk.org> wrote:

> 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.

Ok. But the change what I had for LoginContext.java in this commit was not appropriate : https://github.com/openjdk/jdk/pull/9159/commits/13a51a6d2e026225ec8ac9f1516c7f709c72ce08 right?

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

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