deac/gitolite
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

rmrepo alternative 2 was too lax

Browse source 
it would have allowed deleting a non-wild repo too!  Fixed.
This commit is contained in:
Sitaram Chamarty 2010-09-03 08:48:51 +05:30
parent 5188ec48ed
commit 07d07cc5f6
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
contrib/adc/rmrepo
View file

@ -13,6 +13,8 @@ get_rights_and_owner $delete
# alternative 2 -- creator or anyone with W access to admin repo can delete # alternative 2 -- creator or anyone with W access to admin repo can delete
[ "$owner" = "$GL_USER" ] || { [ "$owner" = "$GL_USER" ] || {
# but only if it is wild
[ "$owner" = "<gitolite>" ] && die "$delete is not a wild repo"
get_rights_and_owner gitolite-admin get_rights_and_owner gitolite-admin
[ -z "$perm_write" ] && die "$delete is not yours to delete and you're not the admin" [ -z "$perm_write" ] && die "$delete is not yours to delete and you're not the admin"
} }