search

ClusterCockpit / cc-backend

Web frontend and API backend server for ClusterCockpit Monitoring Framework
https://www.clustercockpit.org
MIT License
14 stars 12 forks source link

Ldap name config #207

Closed giesselmann closed 10 months ago

giesselmann commented 10 months ago

Hi, I would like to propose a small improvement to the ldap auth module: In our LDAP, full user names are stored in the 'cn' attribute, not in 'gecos'. I made the attribute to look for user names a config option which defaults to 'gecos' if not provided. Shouldn't break with older configs like this.

Also fixes a small bug in DelUser where the error log was printed independent of the error.