kallithea Changeset - e27ff6a90076

Changeset - e27ff6a90076

Parent rev.

Child rev.

[Not reviewed]

stable

0 1 0

Mads Kiilerich (mads) - 3 years ago 2023-05-07 18:19:38
mads@kiilerich.com

auth: always consider the repo group owner an admin when computing it's permissions

When computing repo group permissions in repository_group_permissions(), always
give admin permissions to the group owner.

That is similar to how repository_permissions() gives admin permissions to the
repo owner.

The extra computation shouldn't cause any extra database hits or make the
computation more complex or expensive, so that should be fine for stable.

Note: This will leave behind some (automaticly added) explicit permissions. I
consider this a very minor glitch, not worth addressing.

1 file changed with 4 insertions and 1 deletions:

kallithea/lib/auth.py

0 comments (0 inline, 0 general)

kallithea/lib/auth.py

➞

Show inline comments

@@ @@ -262,7 +262,10 @@ class AuthUser(object): @@
             # on given group
             for perm in default_repo_groups_perms:
                 rg_k = perm.group.group_name
                 p = perm.permission.permission_name
                 if perm.group.owner_id == self.user_id:
                     p = 'group.admin'
                 else:
                     p = perm.permission.permission_name
                 repository_group_permissions[rg_k] = p
             # user group for repo groups permissions

0 comments (0 inline, 0 general)