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

Admin allows adding two users with the same username #222

Open
mvinent opened this issue Oct 14, 2024 · 2 comments
Open

Admin allows adding two users with the same username #222

mvinent opened this issue Oct 14, 2024 · 2 comments
Labels
bug Something isn't working question Further information is requested

Comments

@mvinent
Copy link
Contributor

mvinent commented Oct 14, 2024

Describe the bug
The admin allows adding two users with the same username, which should not be permitted and also leads to errors. This means not being able to acces with that username even trying with both passwords created for each instance of the username.

To Reproduce
Steps to reproduce the behavior:

  1. Go to admin
  2. Go to user screen
  3. Create new user with the same name of an existing user.
  4. Try to login with that username
  5. See error

Expected behavior
The admin should check for existing users and not allow repeated usernames. I could show a popup informing the error.

@fjlopez fjlopez added the bug Something isn't working label Oct 15, 2024
@fjlopez
Copy link
Contributor

fjlopez commented Oct 15, 2024

I am unable to replicate this bug. With an existing user with username admin, it is not possible to create a new user with the same username. In the Usuari-Nou form:

  • Usuari (username) is the login name and must be unique.
  • Nom (firstName) is the first name and can have multiple occurrences
Captura de pantalla 2024-10-15 a las 17 59 08

@mvinent, could you please provide an example and a screenshot that demonstrates how to reproduce this bug?

@fjlopez fjlopez added the question Further information is requested label Oct 15, 2024
@mvinent
Copy link
Contributor Author

mvinent commented Oct 16, 2024

You're right in the 'common' deployed app it's working fine, as you say. We've detected this bug in our own deployment of the app, it's possible that we have deployed an older version.

image

We'll donwload the latest version and we'll test it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants