Skip to content

Feature for using different bind user in active directory LDAP authentication provider #3933

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

Closed
wants to merge 1 commit into from

Conversation

MichaelF25
Copy link

This adds a feature in the authentication provider for active directory to use a different user for the bind operation as the user which credentials are to be authenticated.

In some company AD servers it's not allowed for ordinary users to connect via LDAP. Microsoft recommends to use a dedicated machine to machine (M2M) user account for these operations.

This adds a feature in the authentication provider for active directory
to use a different user for the bind operation as the user which credentials
are to be authenticated.
@MichaelF25 MichaelF25 changed the title Feature for using different bind user Feature for using different bind user in active directory LDAP authentication provider Jun 16, 2016
@rwinch rwinch modified the milestone: 4.2.0 M1 Aug 15, 2016
@jgrandja jgrandja self-assigned this Sep 1, 2016
@jgrandja
Copy link
Contributor

Thanks for submitting this PR @MichaelF25

We are currently re-thinking our approach to an Active Directory AuthenticationProvider implementation. Our direction is to deprecate ActiveDirectoryLdapAuthenticationProvider and leverage the existing LdapAuthenticationProvider and provide a custom strategy of LdapAuthenticator for Active Directory.

The work has already started in #4064 if you would like to track the progress and provide any input/feedback.

I'm going to close this PR as we will address your input/feedback in #4064.
Thank you.

@jgrandja jgrandja closed this Sep 19, 2016
@jgrandja jgrandja removed this from the 4.2.0 M1 milestone Sep 19, 2016
@jgrandja jgrandja added the status: declined A suggestion or change that we don't feel we should currently apply label Sep 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants