-
Notifications
You must be signed in to change notification settings - Fork 25.3k
Fix test in AnonymousUserIntegTests #82977
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix test in AnonymousUserIntegTests #82977
Conversation
Pinging @elastic/es-security (Team:Security) |
Most likely this started manifesting when other test methods have been added to the test suite class. In any case, the value should be static for the test suite. |
It is fixed by elastic#82977 Resolves: elastic#82971
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All test cluster nodes should be using the same value for setting ANONYMOUS_AUTHORIZATION_EXCEPTION_SETTING.
* Fix test in AnonymousUserIntegTests #82977 All test cluster nodes should be using the same value for setting ANONYMOUS_AUTHORIZATION_EXCEPTION_SETTING. * [Test] Fix header value assertion for 401 error (#83037) The WWW-Authenticate header is multi-valued. In rare cases, the first value may not be the one beginning with "Basic". The PR makes the assertion agnostic to the order and also asserts for all possible header values. Resolves: #83022 Co-authored-by: Albert Zaharovits <albert.zaharovits@elastic.co> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
All nodes should be using the same setting for
AuthorizationService.ANONYMOUS_AUTHORIZATION_EXCEPTION_SETTING
.Fixes https://gradle-enterprise.elastic.co/s/5ms2ktf4h5s3i/console-log?task=:x-pack:plugin:security:internalClusterTest