Using username in filter

[shaneholder]

I'm looking to use passport-ldap in a project and I wanted to make sure I understand something, it looks like the user/password are bound but then all entries in the searchFilter are returned and not just the entry associated with the user who was authenticated, is this correct? Is there a reason to not use the username in the filter?

Thanks,
Shane

[shaneholder]

I just saw the latest PR and saw that the code merge to support uid substitution but I'm not sure it's correct. It seems to replace the original search filter with the substitution, and then subsequent substitutions use the original replaced value. Perhaps I'm missing something?

      self._options.search.filter = self._options.search.filter.replace(/\$uid\$/, req.body.username);

also replacing req.body.username with req.body.user seems to not be inline with passport functionality.

Thanks,
Shane.

[mintbridge]

does the example help with this?