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]: Unable to log in after registering an account with the correct credentials #4413

Closed
bimsina opened this issue Apr 11, 2024 · 7 comments
Labels
🐛 bug Something isn't working high priority

Comments

@bimsina
Copy link

bimsina commented Apr 11, 2024

Describe the bug

Unable to log in after registering an account with the correct credentials. The system returns the following error message:

{
    "code":"INVALID_CREDENTIALS",
    "params":null
}

To Reproduce

Steps to reproduce the behavior:

  1. Clone the repository.
  2. Execute sh tools/deploy.sh to generate a .env file.
  3. Use the command docker compose -p activepieces up to start the services.
  4. Register a new account successfully.
  5. Log out from the newly created account.
  6. Register another account using a different email address.
  7. Log out again.
  8. Attempt to log in with the credentials of the second account results in the error mentioned above.

Expected behavior

The expectation is that the account should log in successfully when the correct credentials are provided.

Additional context

From my initial analysis, it appears that a new platformId is generated for each registration. During the login process, the system seems to fail in locating the corresponding platformId, resulting in an "user not found" scenario. This might be the root cause of the login issues encountered.

@abuaboud abuaboud added the 🐛 bug Something isn't working label Apr 11, 2024
@abuaboud
Copy link
Contributor

Hi @bimsina

Good catch, let me try to take look at that

@abuaboud
Copy link
Contributor

@allcontributors please add @bimsina for bug

Copy link
Contributor

@abuaboud

I've put up a pull request to add @bimsina! 🎉

@bimsina
Copy link
Author

bimsina commented Apr 23, 2024

@abuaboud Any updates on this issue?

@abuaboud
Copy link
Contributor

abuaboud commented Apr 23, 2024

Yes, I am waiting for this to land: #4481, to start progressing on the issue.

Now, the issue here is that we need the platform ID to authenticate the user. Usually, we retrieve it from the custom domain, or in the community edition, we simply get the oldest platform.

When the user logs in, the email is only unique across same platform at the moment, which means we can't determine which platform the user belongs to. However, I think in the community edition, we can make the email unique across instance, and that's way we can reverse-map to the platform ID.

I am hoping to finish all work related to this change as soon as possible so we can land it.

@abuaboud
Copy link
Contributor

I am closing this issue, It will be fixed in the next release (after 0.25.1) It's already merged in main.

#4563

Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If this issue is continuing with the latest stable version of Activepieces, please open a new issue that references this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working high priority
Projects
None yet
Development

No branches or pull requests

2 participants