Skip to content

Navigation Menu

Sign in
Sign up

Allow rewrite of UID for SSO - #313

Open
digitalshow wants to merge 2 commits into
nextcloud:master from
digitalshow:master
Open

Allow rewrite of UID for SSO #313
digitalshow wants to merge 2 commits into
nextcloud:master from
digitalshow:master

Conversation

@digitalshow

@digitalshow digitalshow commented Mar 1, 2019

Copy link
Copy Markdown

Allow for rewrite of UIDs using preg_replace.

An administrator can provide a pattern and a replacement parameter that get passed to PHP's preg_replace function before the UID is used to sign in to NextCloud. I use it to extract the username from a kerberos principal (pattern: /^(.*)@REALM$/, replacement: 1ドル).

hibby50 and CorentinS6 reacted with thumbs up emoji

Copy link
Copy Markdown
Member

thanks for your contribution! 👍 Sounds like a good idea but I don't have a Kerberos setup to test it at the moment... @icewind1991 do you have a chance to review/test it?

Copy link
Copy Markdown
Author

It is not limited to Kerberos, you can use it in other places where the username has to be rewritten (e.g. when the SSO mechanism passes an e-mail address or a full path from LDAP).

Copy link
Copy Markdown
Author

Any news here?

Copy link
Copy Markdown
Member

@digitalshow thanks for this pr!
Could you rebase please? :)

Signed-off-by: Ingo Koinzer <ingo@koinzer.net>
Signed-off-by: Ingo Koinzer <ingo@koinzer.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@icewind1991 icewind1991 Awaiting requested review from icewind1991
@rullzer rullzer Awaiting requested review from rullzer
@blizzz blizzz Awaiting requested review from blizzz blizzz is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /