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

LDAP Authenticator does not allow multiple bind template #21

Closed
mrinmoyprasad opened this issue Sep 27, 2016 · 8 comments
Closed

LDAP Authenticator does not allow multiple bind template #21

mrinmoyprasad opened this issue Sep 27, 2016 · 8 comments

Comments

@mrinmoyprasad
Copy link

c.LDAPAuthenticator.bind_dn_template configuration only takes one value? I have my users spread across multilple OUs

For Eg: 'cn={username},ou=Developers,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=com'
'cn={username},ou=Analysts,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=com'
'cn={username},ou=Bigdata,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=com'

Is there a way I can use multiple bind template in this configuration?

@yuvipanda
Copy link
Collaborator

Hello! This sounds like a very useful feature! I won't have any time to
work on it for at least 2 weeks, but will happily review and merge patches!
Do you think you can take a shot at implementing it?

Thanks!

On Tue, Sep 27, 2016 at 7:04 AM, mrinmoyprasad notifications@github.com
wrote:

c.LDAPAuthenticator.bind_dn_template configuration only takes one value?
I have my users spread across multilple OUs

For Eg: 'cn={username},ou=Developers,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=
com'
'cn={username},ou=Analysts,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=com'
'cn={username},ou=Bigdata,ou=All Users,dc=abc,dc=ds,dc=xyz,dc=com'

Is there a way I can use multiple bind template in this configuration?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#21, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAB23g4krECw1Fg8ThzkRTV2TPVR4IwDks5quSJXgaJpZM4KHtLX
.

Yuvi Panda T
http://yuvi.in/blog

@kishorchintal
Copy link
Contributor

I have a solution implemented for this, will fork and submit a pull request. Thanks.

@kishorchintal
Copy link
Contributor

kishorchintal commented Sep 28, 2016

Submitted a pull request #23 Support for multiple bind templates

@dantchapman
Copy link

I'm trying to make use of the multiple c.LDAPAuthenticator.bind_dn_template configuration - however this does not appear to be present in the standard pip install method. How should I go about using this newer feature?

@yuvipanda
Copy link
Collaborator

yuvipanda commented Apr 6, 2017 via email

@dantchapman
Copy link

Thanks for this - I installed the latest locally and it is working. Would be great to get a release so I can pass to our admins.

@nicolajthulin
Copy link

nicolajthulin commented Feb 23, 2018

I use this form c.LDAPAuthenticator.bind_dn_template = '{domain_name} \ {username}'

@dhirschfeld
Copy link
Collaborator

Closing as this is supported in master so will be available in the next release - #70

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

No branches or pull requests

7 participants