Skip to content

Remove JWT realm feature flag.#84845

Merged
justincr-elastic merged 11 commits intoelastic:masterfrom
justincr-elastic:feature/jwt-realm-remove-feature-flag
Mar 24, 2022
Merged

Remove JWT realm feature flag.#84845
justincr-elastic merged 11 commits intoelastic:masterfrom
justincr-elastic:feature/jwt-realm-remove-feature-flag

Conversation

@justincr-elastic
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

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

@justincr-elastic
Copy link
Copy Markdown
Contributor Author

@elasticsearchmachine update branch

1 similar comment
@justincr-elastic
Copy link
Copy Markdown
Contributor Author

@elasticsearchmachine update branch

@justincr-elastic justincr-elastic marked this pull request as draft March 10, 2022 01:13
@justincr-elastic
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Strange Gradle 8.0 error. Retrying build.

@justincr-elastic
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@justincr-elastic
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

@elasticmachine update branch

@justincr-elastic
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@justincr-elastic
Copy link
Copy Markdown
Contributor Author

It is ready for review.

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

Thank you.

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

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

Development

Successfully merging this pull request may close these issues.

5 participants