Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Duplicate entry related to user role association #145

Open
shukla-m opened this issue Apr 11, 2024 · 1 comment
Open

Duplicate entry related to user role association #145

shukla-m opened this issue Apr 11, 2024 · 1 comment
Assignees
@shukla-m
Labels
🐛 bug Something isn't working ⬇️ priority:low Low priority issue

Comments

@shukla-m
Copy link

See Sentry error PROFILES-63.

Illuminate\Database\QueryException

SQLSTATE[23000]: Integrity constraint violation: Duplicate entry for key 'role_user.PRIMARY' (SQL: insert into role_user (role_id, user_id) values (*, ****))
@shukla-m shukla-m added 🐛 bug Something isn't working ⬇️ priority:low Low priority issue labels Apr 11, 2024
@shukla-m shukla-m self-assigned this Apr 11, 2024
@shukla-m shukla-m changed the title Duplicate entry issue related to user role association Duplicate entry related to user role association Apr 11, 2024
@wunc wunc added the 👥 duplicate This issue or pull request already exists label Apr 23, 2024
@wunc
Copy link
Collaborator

wunc commented Apr 23, 2024

Related to #92. (Not really duplicate, but it's an error that would likely be mitigated by implementing a submit button disable). Another solution is to maybe refactor the LdapAttributeHandler@syncUserRoles() method to more gracefully handle mismatched roles (maybe cached?) between closely-spaced requests.

@wunc wunc removed the 👥 duplicate This issue or pull request already exists label Apr 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@shukla-m shukla-m

Labels
🐛 bug Something isn't working ⬇️ priority:low Low priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wunc @shukla-m