kitware-resonant / dkc-next

Apache License 2.0
3 stars 0 forks source link

Allow explicit permissions to be added for superusers #121

Closed jbeezley closed 3 years ago

jbeezley commented 3 years ago

This fixes something that could be seen as a bug where permissions weren't actually being added for superusers. There is no behavioral change here because superusers can always do anything anyway.

Alternatively, we could make it so superusers are always included in the list of permissions returned by GET /folders/{id}/permissions. This has the problem that the implicit super user permissions couldn't be removed, and it would expose a global list of superusers to everyone.