Skip to content

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

Merged
elasticsearchmachine merged 1 commit intoelastic:8.6from
ywangd:backport/8.6/pr-92221
Dec 12, 2022
Merged

[8.6] Support stored authentication headers prior to version 6.7 (#92221)#92295
elasticsearchmachine merged 1 commit intoelastic:8.6from
ywangd:backport/8.6/pr-92221

Conversation

@ywangd
Copy link
Copy Markdown
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-without-approval 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-without-approval 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.

2 participants