Skip to content

Loading…

Login search filter #10

Open
AngryAngryHippos opened this Issue · 0 comments

1 participant

@AngryAngryHippos

I want to do a login search filter by multiple fields, like this:

'(|(sAMAccountName={{username}})(mail={{username}}))'

When I try it, it only allows login by the first critera, in this case, sAMAccountName. I narrowed the bug down to ldapauth.js line 118 - just needs to be a global search and replace:

var searchFilter = self.opts.searchFilter.replace(/{{username}}/g, username);

@vesse vesse added a commit to vesse/node-ldapauth-fork that referenced this issue
@vesse vesse Use global search/replace for {{username}}
Use global search/replace to allow eg. OR filters
(see trentm/node-ldapauth#10)
098f0d1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.