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

Support unauthenticated and anonymous bind #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Support unauthenticated and anonymous bind #12

wants to merge 1 commit into from

Conversation

danielm0hr
Copy link

LDAP allows three authentication modes when using
Simple Authentication for binding: Authenticated
(user, password), unauthenticated (only user) and
anonymous (no user, no password). See RFC4513
Section 5.1 for reference.

This adds support for all three modes by updating
to the newest version of go-ldap/ldap and handling
the ldap-search-user-dn and ldap-search-user-password
flags appropriately.

Signed-off-by: Daniel Mohr [email protected]

@rajatjindal
Copy link

Hi Daniel,

I tried reaching out to the owners of the project, but it looks like its not actively maintained.

we have a fork that we are enhancing here: proofpoint/kubernetes-ldap

@danielm0hr
Copy link
Author

Hi @rajatjindal,

thanks for the information. I see that you also have a Dockerfile there. That's one of the things I wanted to add next. Do you also push the image to Docker Hub? Couldn't find it there.

@rajatjindal
Copy link

Yes that would be done sometime this week or early next week

@danielm0hr
Copy link
Author

Ok, then let me rebase my stuff on top of your fork and create pull requests there...

LDAP allows three authentication modes when using
Simple Authentication for binding: Authenticated
(user, password), unauthenticated (only user) and
anonymous (no user, no password). See RFC4513
Section 5.1 for reference.

This adds support for all three modes by updating
to the newest version of go-ldap/ldap and handling
the ldap-search-user-dn and ldap-search-user-password
flags appropriately.

Signed-off-by: Daniel Mohr <[email protected]>
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.

2 participants