Skip to content

Remove deprecated RealmConfig constructor#35327

Merged
tvernum merged 2 commits intoelastic:masterfrom
tvernum:realm-settings-cleanup
Nov 7, 2018
Merged

Remove deprecated RealmConfig constructor#35327
tvernum merged 2 commits intoelastic:masterfrom
tvernum:realm-settings-cleanup

Conversation

@tvernum
Copy link
Copy Markdown
Contributor

@tvernum tvernum commented Nov 7, 2018

This removes an obsolete constructor that was still being called from
some tests.

Relates: #30241

This removes an obsolete constructor that was still being called from
some tests.

Relates: elastic#30241
@tvernum tvernum added v7.0.0 >refactoring :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) labels Nov 7, 2018
@tvernum tvernum requested a review from bizybot November 7, 2018 04:08
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-security

Copy link
Copy Markdown
Contributor

@bizybot bizybot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks.

@tvernum tvernum merged commit b4173c8 into elastic:master Nov 7, 2018
pgomulka pushed a commit to pgomulka/elasticsearch that referenced this pull request Nov 13, 2018
This removes an obsolete constructor that was still being called from
some tests.

Relates: elastic#30241
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>refactoring :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants