seek4science / seek

For finding, sharing and exchanging Data, Models, Simulations and Processes in Science.
http://www.seek4science.org
BSD 3-Clause "New" or "Revised" License
76 stars 52 forks source link

Improper redirection to `/people/register` #843

Open ilveroluca opened 2 years ago

ilveroluca commented 2 years ago

Describe the bug Trying to Log In doesn't abort failed creation of a new profile.

To Reproduce Steps to reproduce the behavior:

  1. Log into Workflow Hub with ELIXIR AAI.
  2. Fill in the fields to create a New Profile. Insert the required information and the email of your existing WorkflowHub account.
  3. The system gives an error saying that "Email has already been taken".
  4. Click "Log In" on the upper right
  5. Click "Sign out"

The system redirects me to the New Profile page and I can't Log In until I click on the link to cancel the creation of the new profile.

Expected behavior In the specific scenario described above, I would expect to have a chance to merge my ELIXIR AAI and WorkflowHub accounts. More in general, I would expect that clicking on "Log In" should abort the creation of the new profile.

Desktop (please complete the following information):

Smartphone (please complete the following information):

fbacall commented 2 years ago

Hi, we can't merge accounts based on email address because ELIXIR AAI does not guarantee the veracity of the email address it reports back to us.

I agree that being stuck in the registration process is awkward though. We can look into aborting the process if the user tries to log in.

fbacall commented 2 years ago

Also you can add ELIXIR AAI as a log-in method to your original account by going to the "Manage identities" page from the top-right menu