Skip to content

Commit

Permalink
Update FIXME in ScimUserController
Browse files Browse the repository at this point in the history
Co-authored-by: jordigh <[email protected]>
  • Loading branch information
fflorent and jordigh authored Nov 29, 2024
1 parent 8312aa9 commit 4beef42
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app/server/lib/scim/v2/ScimUserController.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,8 @@ class ScimUserController {
throw new SCIMMY.Types.Error(403, null!, 'Technical user deletion not permitted.');
}
const fakeScope: Scope = { userId: id };
// FIXME: deleteUser should probably better not requiring a scope.
// FIXME: deleteUser should probably be rewritten to not require a scope. We should move
// the scope creation to a controller.
await this._dbManager.deleteUser(fakeScope, id);
});
}
Expand Down

0 comments on commit 4beef42

Please sign in to comment.