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

add STARTTLS/certificate check to ldap machine resolvers #638

Closed
jh23453 opened this issue Mar 2, 2017 · 5 comments
Closed

add STARTTLS/certificate check to ldap machine resolvers #638

jh23453 opened this issue Mar 2, 2017 · 5 comments
Assignees
Labels
Topic: LDAP LDAP resolver and LDAP related issues
Milestone

Comments

@jh23453
Copy link
Contributor

jh23453 commented Mar 2, 2017

Hello,

now we have added certificate checks to the user resolver - should we have the same
for the ldap machine resolver in privacyidea/lib/machines/ldap.py? I think yes :-)

@cornelinux
Copy link
Member

Yes, we do!
So you are the guy who is using the machine resoler? ;-)
Tell me more about it. If you have any idea on improving it in regards to groups etc. Please take a look at #285

@jh23453
Copy link
Contributor Author

jh23453 commented Mar 3, 2017 via email

@cornelinux cornelinux changed the title ldap machine resolver: add STARTTLS/certificate check too? add STARTTLS/certificate check to ldap machine resolvers Mar 3, 2017
@cornelinux cornelinux added this to the 2.19 milestone Mar 13, 2017
@cornelinux cornelinux added the Topic: LDAP LDAP resolver and LDAP related issues label Mar 13, 2017
@cornelinux
Copy link
Member

take a look at the ldap user resolver and #639

@fredreichbier
Copy link
Contributor

Looking into this, it seems like the LDAP machine resolver currently does not respect the No anonymous referral chasing option. Analogously to #658, the LDAP machine resolver always disables anonymous referral chasing. But that's an easy fix. :)

fredreichbier pushed a commit that referenced this issue Apr 26, 2017
fredreichbier pushed a commit that referenced this issue Apr 26, 2017
* respect NOREFERRALS parameter
* remove unused EDITABLE parameter

Working on #638
fredreichbier pushed a commit that referenced this issue Apr 26, 2017
cornelinux pushed a commit that referenced this issue Apr 26, 2017
cornelinux pushed a commit that referenced this issue Apr 26, 2017
* respect NOREFERRALS parameter
* remove unused EDITABLE parameter

Working on #638
cornelinux pushed a commit that referenced this issue Apr 26, 2017
@cornelinux
Copy link
Member

Closed by c5c91c4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic: LDAP LDAP resolver and LDAP related issues
Projects
None yet
Development

No branches or pull requests

3 participants