Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

addOptionForLdapOauth #1501

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Conversation

kbilev
Copy link
Contributor

@kbilev kbilev commented May 3, 2023

Hallo,
folgender Use-Case: Unser Reverse Proxy befüllt LDAP Gruppen über Rollen und dient zugleich als oAuth Endpunkt.
Jetzt erscheint es sinnvoll, wenn auch LDAP User sich über Single Sign On einloggen können um die User-Session vom Reverse Proxy mitzubenutzen.
Momentan sehe ich nur die Möglichkeit SSO für lokale Benutzer zu aktivieren. Deshalb hier eine Idee um auch SSO LDAP Usern zur Verfügung zu stellen

@nook24
Copy link
Member

nook24 commented May 30, 2023

Hallo @kbilev

vielen Dank für deinen Pull Request. Dieses Feature ist bereits verfügbar, allerdings fehlt beim "Edit" eines Benutzers das entsprechende Häkchen. Ob das ein Fehler oder gewollt ist, muss noch geklärt werden.

In den Systemsettings kannst du den Wert von AUTH_METHOD auf SSO oAuth2 and LDAP setzen.
Wenn diese Option gesetzt ist, werden nach dem Login eines Benutzers noch die Gruppen vom LDAP abgefragt. Dafür müssen die Benutzer unter Manage Users -> Import from LDAP erstellt werden und der Haken bei Enable login through oAuth2 aktiviert werden.

Bei schon bestehenden Benutzer, kann dies aktuell nachträglich nur über die Datenbank aktiviert werden.

UPDATE users SET is_oauth=1 WHERE samaccountname IS NOT NULL;

Danach stehen für diesen Benutzer beide Login-Methoden zur Verfügung. Dies wird meistens in Kombination mit einem Azure Active Directory genutzt.

LDAP_and_oAuthjpg

Bitte vor einer Änderung an der Datenbank eine Sicherung über Backup / Restore erstellen!

Viele Grüße

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants