Library doesn`t allow to set uid on 1st place

T4cC0re / ldap-verifyuser

ldap-verifyuser is a node.js library based on ldapjs, that authenticates a user against a LDAP or AD without requiring an configured administrator account

MIT License

4 stars 5 forks source link

Library doesn`t allow to set uid on 1st place #2

Open Ohar opened 8 years ago

Ohar commented 8 years ago

Your library save me, but there is a bug (or, maybe, a feature, you know) which doesn’t allow me to send query as uid=LOGIN,ou=Users,dc=FOO,dc=BAR, but force me to put username to the end of the string and that’s why I can’t connect to LDAP properly.

T4cC0re commented 8 years ago

Hi, first of all sorry it took me so long to respond, but thank you for noticing this.

I will take a closer look at this and also think about your commit.

T4cC0re commented 8 years ago

18:09:48 ✓ t4cc0@RainbowDash ~/Repos/ldap-verifyuser @(master)
$ npm publish
+ ldap-verifyuser@1.1.0

Should be fixed in 1.1.0.

Thanks again for reporting. If I have misunderstood your issue, please let me know and reopen.

Ohar commented 8 years ago

@T4cC0re Very much thanks for you collaboration, but I told about other problem: I cant sendusernameto LDAP server without [concating](https://github.com/T4cC0re/ldap-verifyuser/commit/6c67d15e7d428c00e3ed614fdde3696f45997999#diff-887c0c4e437540e383f397836135e5d2R19) it withadrdn`.

Why do you concatinate this two strings? Is it strictly neccessary?

Thank you.