hydrian / TTRSS-Auth-LDAP

GitHub repository for Tiny Tiny RSS's auth_ldap plugin
https://github.com/hydrian/TTRSS-Auth-LDAP/wiki
Other
28 stars 21 forks source link

Include login attribute in list of attributes to fetch #20

Closed MrsKensington closed 8 years ago

MrsKensington commented 8 years ago

We need to provide the login attribute in the list of attributes to fetch, otherwise (if you're not using displayName, title or sAMAccountName for example "uid") as the login attribute it fails later on with "Could not find user name attribute uid in LDAP entry".

To be honest I'm not sure if we need the rest of the list of attributes as I couldn't see when they would be used but wasn't going to delete them and break something unforseen.