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

fix: do never link users with organisation identities #1196

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

open-dynaMIX
Copy link
Member

@open-dynaMIX open-dynaMIX commented Jan 17, 2025

This is the first step to resolve this issue. This fix prevents
organisation Identities to be linked with keycloak users. This could
potentially lead to a situation, where a user can make a keycloak
account with their email address, but then be unable to login to mySAGW,
if the Email is already set on an organisation Identity, due to the unique
constraint on the email field.

@open-dynaMIX open-dynaMIX changed the title fix: add command to move organisation emails into their own Email object fix: remove emails from organisation identities Jan 17, 2025
@open-dynaMIX open-dynaMIX force-pushed the move_organisation_email_command branch 2 times, most recently from af09bfc to 2a9682e Compare January 22, 2025 08:51
@open-dynaMIX open-dynaMIX changed the title fix: remove emails from organisation identities fix: do never link users with organisation identities Jan 22, 2025
@open-dynaMIX open-dynaMIX marked this pull request as ready for review January 22, 2025 10:17
@open-dynaMIX open-dynaMIX force-pushed the move_organisation_email_command branch 2 times, most recently from 5f85ab5 to 528cc8b Compare January 23, 2025 13:12
This is the first step to resolve this issue. This fix prevents
organisation Identities to be linked with keycloak users. This could
potentially lead to a situation, where a user can make a keycloak
account with their email address, but then be unable to login to mySAGW,
if the Email is already set on an organisation Identity, due to the unique
constraint on the email field.
@open-dynaMIX open-dynaMIX force-pushed the move_organisation_email_command branch from 528cc8b to a7b862d Compare January 23, 2025 13:13
@Yelinz Yelinz force-pushed the move_organisation_email_command branch from 58ecaca to d6281f2 Compare January 23, 2025 14:12
redirect to support page when receiving specific identity error
@Yelinz Yelinz force-pushed the move_organisation_email_command branch from d6281f2 to e41cbc7 Compare January 23, 2025 14:48
@Yelinz Yelinz requested a review from velrest January 23, 2025 14:51
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.

3 participants