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

[ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so… #4444

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

paul8263
Copy link
Contributor

…me wrong config items.

What is this PR for?

Changed the configuration in LdapRealm example from
ldapRealm.contextFactory.systemAuthenticationMechanism=simple to ldapRealm.contextFactory.authenticationMechanism=simple.

And enriched the explanation for userSearchAttributeName and memberAttributeValueTemplate, which might be hard to understand before this PR.

What type of PR is it?

Documentation

What is the Jira issue?

ZEPPELIN-5799

Questions:

  • Does the licenses files need to update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@paul8263 paul8263 changed the title Enriched LdapRealm configuration in Shiro Authentication and fixed so… [ZEPPELIN-5799] Enriched LdapRealm configuration in Shiro Authentication and fixed so… Aug 16, 2022
@paul8263
Copy link
Contributor Author

paul8263 commented Aug 18, 2022

Hi @jongyoul ,

Coul you please help rerun the tests? This PR is only for docs and leave those tests unchanged. They should not fail. Thanks.

@jongyoul
Copy link
Member

@paul8263 Yes, correct. I just would like to execute license-check. Please don't care about others. :-)

@jongyoul jongyoul merged commit 71c97ac into apache:master Aug 18, 2022
@Reamer
Copy link
Contributor

Reamer commented Aug 22, 2022

With this PR we have defined the same property twice.
ldapRealm.contextFactory.authenticationMechanism

@Reamer
Copy link
Contributor

Reamer commented Aug 22, 2022

PR: #4447

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants