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

Check user belongs to organization before update #455

Merged
merged 2 commits into from
Sep 30, 2022

Conversation

florimondmanca
Copy link
Collaborator

Suite de #437

Closes #388

Cette PR vérifie que l'utilisateur qui fait la requête de mise à jour d'un jeu de données a bien le droit de le faire, = appartient à l'organisation à laquelle le jeu de données est rattaché.

@florimondmanca florimondmanca merged commit 776b141 into fm/dataset-perms Sep 30, 2022
@florimondmanca florimondmanca deleted the fm/dataset-perms-update branch September 30, 2022 10:30
florimondmanca added a commit that referenced this pull request Sep 30, 2022
* Check user belongs to dataset organization before creation

* Prevent ADMIN from creating dataset outside own org, refactor tests

* Refine

* Check user belongs to organization before update (#455)

* Check user belongs to organization before update

* Test ADMIN cannot update in other org either
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants