Skip to content

bug(SCIM) : cannot delete user with a team site #1540

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

Open
1 of 2 tasks
vviers opened this issue Mar 28, 2025 · 0 comments
Open
1 of 2 tasks

bug(SCIM) : cannot delete user with a team site #1540

vviers opened this issue Mar 28, 2025 · 0 comments
Assignees
Labels
bug Something isn't working gouv.fr

Comments

@vviers
Copy link
Collaborator

vviers commented Mar 28, 2025

Describe the current behavior

The SCIM API implementation in #1199 lets a superadmin (GRIST_DEFAULT_EMAIL) hard-delete a user on the instance via the DELETE /scim/v2/Users/{userId} endpoint.

However, this does not work if the user to be deleted has a team site in addition to his Personnal site. The app returns a 500 internal error.

Steps to reproduce

  1. create a user
  2. create a team site
  3. connect as a superadmin
  4. try to delete the user

Describe the expected behavior

Superadmins should be able to delete a user on the instance they manage regardless of whether the user has created a team site or not.

In our context, the expected behavior would be :

  • if the user is the only owner of an org, delete the org
  • if there are other owners of an org, simply remove the user from the org before deleting the user

Where have you encountered this bug?

Instance information (when self-hosting only)

  • Grist instance:
    • Version: 1.4.2
    • Installation mode: kubernetes
    • Architecture: multi-workers
@vviers vviers added bug Something isn't working gouv.fr labels Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gouv.fr
Projects
None yet
Development

No branches or pull requests

2 participants