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

clarify "unlink users" action #29086

Closed
wants to merge 1 commit into from
Closed

clarify "unlink users" action #29086

wants to merge 1 commit into from

Conversation

IdeoG
Copy link

@IdeoG IdeoG commented Apr 25, 2024

Hey community,

We recently unlinked all users from an LDAP user federation and got in trouble, when we lost all assigned user roles on LDAP users after the "unlink users" action.

Since I didn't find any information about it, I would like to share my experience with you, just to help other people eliminate that kind of unwanted behavior.

Best wishes,
Aleksei.

@IdeoG IdeoG requested a review from a team as a code owner April 25, 2024 14:46
@edewit
Copy link
Contributor

edewit commented Apr 26, 2024

Hi @IdeoG thanks for your contribution, but you need to sign-off the commit, https://github.com/keycloak/keycloak/pull/29086/checks?check_run_id=24257048250

@ssilvert
Copy link
Contributor

ssilvert commented May 2, 2024

Hi @IdeoG thanks for your contribution, but you need to sign-off the commit, https://github.com/keycloak/keycloak/pull/29086/checks?check_run_id=24257048250

@IdeoG You just need to do git commit --amend --signoff and push it again.

@ssilvert
Copy link
Contributor

ssilvert commented May 7, 2024

@IdeoG Unfortunately, we can not accept this without your signoff. I'm going to close this now but we can reopen later when you are ready.

@ssilvert ssilvert closed this May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants