On Thu, 12 Jun 2025 14:56:28 GMT, p-nima <d...@openjdk.org> wrote:

>> Agreed. And it's very good to have a test now to verify the expected 
>> behavior. So I would change the test to test the behaviour we want. Then 
>> double check that it fails with the current proposed changes, and pass if we 
>> revert those changes. Then I would clarify the documentation of the system 
>> property, in order to remove the confusion.
>
> Thanks all for your inputs, I will update the AuthenticationFilter and test 
> accordingly and update the documentation to clarify the confusion.

The changes have been made in 9e8f29ee3ab03f26bbd39b70a6d829ffee2aaa11 - and 
java docs has been updated accordingly to reflect the behaviour.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25490#discussion_r2143079910

Reply via email to