server icon indicating copy to clipboard operation
server copied to clipboard

Updated LDAP Integration

Open gentoo9ball opened this issue 3 years ago • 4 comments

I've taken the older LDAP Pull Request and merged it with the latest code. It is working great in my environment.

From the old code, I've found that the popular kpeiruza/hashtopolis docker image uses the User.class constructor with 16 arguments, the extra argument breaks this. So I've reworked the constructor flow to use either 16 or 17 arguments.

Also, I've made the BaseDN and UID config settings, so this can work in different environments. Some LDAP Servers use CN by default, others use UID. Setting these variables right now has to occur manually via the database.

A couple nice things that I can work on in the future, would be:

  1. A way to specify TLS Certs for LDAPS
  2. A configuration interface to set LDAP Configuration settings (ldap_server, ldap_basedn, ldap_uid)
  3. A way to sync users from LDAP given an admin user

gentoo9ball avatar Feb 07 '22 22:02 gentoo9ball

Still using this on our side, working well

gentoo9ball avatar Apr 06 '22 17:04 gentoo9ball

Is there any progress on this topic? LDAP authentication would be a great feature

Prototype82 avatar Dec 11 '22 16:12 Prototype82

No progress on this has been made, but in the background we have been working on a new frontend and backend. This will come with the switch to oAuth for authentication, which would make it possible to use all kinds of authentication servers like LDAP or AD based servers for authentication.

zyronix avatar Dec 11 '22 18:12 zyronix

Any status update on this? Super useful.

gary-sixgen avatar Jul 10 '23 13:07 gary-sixgen