-
Notifications
You must be signed in to change notification settings - Fork 280
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
lookupLoginByDN complies to filters #14
base: master
Are you sure you want to change the base?
Conversation
…nfigured filters are applied for each source, so we know which cofigured source this entry comes from and which other configuration entries apply.
60e3c1c
to
36b833c
Compare
|
…-when-non-decoding-quoted-printable Fix memory leak when failing to parse a quoted printable
…tions Remove handleException on requests
Any news on this PR? |
@WoodySlum: Why it is closed? |
PR has been automatically closed by bot (too old). I reopen this one, can you @tatankat update the branch ? |
I don't have an update, but I am on a longlingering project of updating our instance. So an update should come one day, if not done by someone else before. |
e4c98fd
to
2258d5c
Compare
Use provided LDAP search method to get login from dn. This way the configured filters are applied for each source, so we know which configured source this ldap entry comes from and which other configuration entries apply (like UIDFieldName).