darkkirb/harness-drone
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fixed issue when no gitlab permissions exist, but user is repo owner

Browse source
This commit is contained in:
Brad Rydzewski 2014-09-03 01:23:45 -07:00
parent eddc11130a
commit c858c3b2b8
1 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
plugin/remote/gitlab/gitlab.go
View file

@ -83,13 +83,23 @@ func (r *Gitlab) GetRepos(user *model.User) ([]*model.Repo, error) {
// if no permissions we should skip the repository
// entirely, since this should never happen
if item.Permissions == nil {
if repo.Owner != user.Login && item.Permissions == nil {
continue
}
repo.Role.Admin = IsAdmin(item)
repo.Role.Write = IsWrite(item)
repo.Role.Read = IsRead(item)
// if the user is the owner we can assume full access,
// otherwise check for the permission items.
if repo.Owner == user.Login {
repo.Role = new(model.Perm)
repo.Role.Admin = true
repo.Role.Write = true
repo.Role.Read = true
} else {
repo.Role.Admin = IsAdmin(item)
repo.Role.Write = IsWrite(item)
repo.Role.Read = IsRead(item)
}
repos = append(repos, &repo)
}