-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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 Auth broken by security groups with parenthesis in the name... #1163
Comments
I figured out what the issue was. The account was part of some AD groups that had parenthesis in their names. Apparently it was referencing that group as a search filter. Since the user was part of the |
Thanks for reporting and doing the investigation work and providing details @derek-shnosh, Really helpful stuff. |
Added a fix in 26ec1cc, Will be in next release. Hoping that will be later this weekend. |
Excellent! |
@ssddanbrown - figured I'd ask here, is there a way to run from the master branch? If I checkout the master branch and refresh the page, it's pretty broken. I'm guessing because public\dist doesn't exist as you mention here. |
@derek-shnosh there is but you'll need a recent version of Node.JS installed and then following the build steps towards the top of this section in the readme. Alternatively, since I've got the files open, here's a zip of built files from current master: Just a warning, master can be a little unstable, Might be okay right now but I'd avoid updates and switch back to release when you can. Next release is a little late, Now hoping to get it out before wednesday the 2nd. |
Thanks for the reponse/input @ssddanbrown. Try to enjoy the weekend and new year celebrations! |
Describe the bug
LDAP authentication seems to succeed, but throws an exception; i.e. a
ldap_search(): Search: Bad search filter
error is given, but if I simply browse back I am logged in and given admin privileges according to the External Authentication ID mapped to the role..env
Steps To Reproduce
Login using AD credentials.
Expected behavior
User logs in and authenticates.
Screenshots
If applicable, add screenshots to help explain your problem.
Your Configuration (please complete the following information):
Additional context
Possibly related: #317
The text was updated successfully, but these errors were encountered: