search

ldapjs / node-ldapjs

LDAP Client and Server API for node.js
http://ldapjs.org
MIT License
1.61k stars 449 forks source link

Update filter documentation #909

Closed jsumners closed 1 year ago

jsumners commented 1 year ago

The substring filter documentation is inconsistent with the current implentation. It should be updated.

The latter are also not mentioned in the example.

The documentation should be updated.

Source: https://github.com/ldapjs/node-ldapjs/issues/907

devbysn commented 1 year ago

Hello @jsumners I've updated the parameters and created a pull request. Can you verify?

devbysn commented 1 year ago

I am still very beginner, i hope i didn't mess anything :)

jsumners commented 1 year ago

This is being resolved by https://github.com/ldapjs/filter/pull/5.