Skip to content
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

[8.6] Support stored authentication headers prior to version 6.7 (#92221) #92295

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Dec 12, 2022

Backports the following commits to 8.6:

…2221)

Officially Elasticsearch is compatible with last major at data level.
Therefore v8 is not compatible with v6. However we don't have a guided
migration path for stored authentication headers, e.g. upgrade assistant
does not do anything for them. Therefore it is more helpful and user
friendly for v8 to support v6 stored authentication headers.

This PR adds back the version conditional logic removed in elastic#41185 along
with tests.
@ywangd ywangd added :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) >bug auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Security Meta label for security team labels Dec 12, 2022
@elasticsearchmachine elasticsearchmachine merged commit 24b2485 into elastic:8.6 Dec 12, 2022
@ywangd ywangd deleted the backport/8.6/pr-92221 branch December 12, 2022 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team v8.6.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants