-
Notifications
You must be signed in to change notification settings - Fork 883
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
[KYUUBI-128]add ldap authenticate support #131
Conversation
Codecov Report
@@ Coverage Diff @@
## master #131 +/- ##
===========================================
+ Coverage 73.01% 73.3% +0.29%
- Complexity 545 551 +6
===========================================
Files 76 77 +1
Lines 3335 3413 +78
Branches 485 509 +24
===========================================
+ Hits 2435 2502 +67
- Misses 638 643 +5
- Partials 262 268 +6
Continue to review full report at Codecov.
|
Hi @dockerzhang, would you please help me test this pull request in your ldap environment? thanks. |
@yaooqinn This pr solved my problem, it works fine in my ldap environment. Thanks again. |
@yaooqinn Could you merge all changes to master? |
thanks @dockerzhang, pending on travis |
No description provided.