-
Notifications
You must be signed in to change notification settings - Fork 59
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 implementation #237
base: master
Are you sure you want to change the base?
ldap implementation #237
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #237 +/- ##
==========================================
+ Coverage 85.90% 85.93% +0.03%
==========================================
Files 47 48 +1
Lines 3902 3968 +66
==========================================
+ Hits 3352 3410 +58
- Misses 550 558 +8 ☔ View full report in Codecov by Sentry. |
I take inspiration from #250 to implement LDAP
|
7f36d89
to
ff5bbc9
Compare
Hi I have some issue for write test do you have any idea on how to do a ldap mock? |
I wish there was some kind of embeddable directory for LDAP, kinda like SQLite... Maybe you can use the unittest.mock library to patch the |
9cfdd80
to
9405f0f
Compare
hello, I have added some test thanks for your advice!! PS: fail test seem not link with my modification |
ca23003
to
187c513
Compare
Hello. Sorry I hadn't taken the time to respond yet. This is something I'd like to try with a real LDAP backend but I lack knowledge in the matter. I'll get back to you as soon as I manage to set up a LDAP server. |
Hi if you want try with fast LDAP instalation you could take a look on LLDAP side |
Hello. I actually managed to setup a basic OpenLDAP server some time ago. I remember noticing uncaught errors if the server wasn't setup properly but I don't recall if it was with your PR or #250. |
Hi like request in #181 I will try to work on LDAP integration I create this pull request for following