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

[Bug]: OAuth: User cannot log in through different orgs #1284

Open
1 task done
jnewman-eti opened this issue Nov 6, 2024 · 3 comments
Open
1 task done

[Bug]: OAuth: User cannot log in through different orgs #1284

jnewman-eti opened this issue Nov 6, 2024 · 3 comments
Assignees
Labels
Java Backend Pull requests that update Java code

Comments

@jnewman-eti
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

When a user logs into lowcoder through oauth provider into one org, logs out, and logs into a different org, this causes an error.

It seems the connection data stored in the db is not handled in a way that facilitates a user to log into different orgs.

Our setup: Keycloak with multiple orgs. Users can exist on both orgs.
Lowcoder Generic Oauth Provider.

Expected Behavior

If a user logs out, and logs back in through a different org, the system should be able to handle this case without causing an error.

Steps to reproduce

Have a multitenant setup in your IDP.
Register the same user in 2 orgs. Same name, email, etc.

Set up the Generic Oauth provider to authenticate through keycloak.
Log into lowcoder using org 1.
Log out of org 1.
Log into org 2.
This should cause an error.

Environment

Lowcoder version 2.4.x
Keycloak IDP

Additional Information

No response

@adnanqaops adnanqaops moved this to 🆕 New in Lowcoder Nov 12, 2024
@FalkWolsky FalkWolsky added the Java Backend Pull requests that update Java code label Nov 18, 2024
@FalkWolsky
Copy link
Contributor

Question... Do you mean Lowcoder Orgs or KeyCloak Orgs (realms)?

dragonpoo added a commit that referenced this issue Nov 22, 2024
@dragonpoo
Copy link
Collaborator

@jnewman-eti
I fixed the issue and it is not reproduced on my side.
Hope you retest it.
Thanks

@dragonpoo
Copy link
Collaborator

After this PR is being merged, of course

ludomikula pushed a commit that referenced this issue Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java Backend Pull requests that update Java code
Projects
Status: 🆕 New
Development

No branches or pull requests

4 participants