Skip to content

Remove JWT realm feature flag. #84845

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 11 commits into from
Mar 24, 2022
Merged

Remove JWT realm feature flag. #84845

merged 11 commits into from
Mar 24, 2022

Conversation

justincr-elastic
Copy link
Contributor

No description provided.

@justincr-elastic justincr-elastic added >feature :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team v8.2.0 labels Mar 9, 2022
@justincr-elastic justincr-elastic requested a review from tvernum March 9, 2022 21:12
@elasticsearchmachine
Copy link
Collaborator

Hi @justincr-elastic, I've created a changelog YAML for you.

@justincr-elastic
Copy link
Contributor Author

@elasticsearchmachine update branch

1 similar comment
@justincr-elastic
Copy link
Contributor Author

@elasticsearchmachine update branch

@justincr-elastic justincr-elastic marked this pull request as draft March 10, 2022 01:13
@justincr-elastic
Copy link
Contributor Author

@elasticsearchmachine update branch

@justincr-elastic justincr-elastic marked this pull request as ready for review March 10, 2022 01:13
@justincr-elastic
Copy link
Contributor Author

Strange Gradle 8.0 error. Retrying build.

@justincr-elastic
Copy link
Contributor Author

@elasticmachine update branch

@justincr-elastic justincr-elastic mentioned this pull request Mar 10, 2022
Merged
@justincr-elastic justincr-elastic marked this pull request as draft March 10, 2022 21:36
@justincr-elastic
Copy link
Contributor Author

Convert to draft because #84740 is adding more usages of the feature flag. This PR #84845 will need to wait for the changes in PR #84740 to be merged.

@tvernum
Copy link
Contributor

tvernum commented Mar 24, 2022

Is this ready for review? We need to move forward with this pretty soon.

@justincr-elastic justincr-elastic marked this pull request as ready for review March 24, 2022 14:52
@justincr-elastic
Copy link
Contributor Author

@elasticmachine update branch

@justincr-elastic
Copy link
Contributor Author

@elasticmachine update branch

@justincr-elastic
Copy link
Contributor Author

It is ready for review.

@justincr-elastic justincr-elastic merged commit 5d331d2 into elastic:master Mar 24, 2022
@justincr-elastic
Copy link
Contributor Author

Thank you.

@tvernum tvernum mentioned this pull request Jul 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@albertzaharovits albertzaharovits albertzaharovits approved these changes

@tvernum tvernum Awaiting requested review from tvernum

Assignees
No one assigned
Labels
>non-issue :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team v8.2.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@justincr-elastic @elasticsearchmachine @tvernum @albertzaharovits @elasticmachine