On Tue, 16 Aug 2022 14:32:46 GMT, Weijun Wang <wei...@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

Reply via email to