mirror of
https://github.com/woodpecker-ci/woodpecker.git
synced 2024-11-26 20:01:02 +00:00
getting gitlab project permissions only if user is not repo owner
also move Timeout value to repo activation
This commit is contained in:
parent
2448c34c6b
commit
617476784d
1 changed files with 5 additions and 13 deletions
|
@ -76,25 +76,12 @@ func (r *Gitlab) GetRepos(user *model.User) ([]*model.Repo, error) {
|
|||
GitURL: item.HttpRepoUrl,
|
||||
SSHURL: item.SshRepoUrl,
|
||||
Role: &model.Perm{},
|
||||
Timeout: 900,
|
||||
}
|
||||
|
||||
if repo.Private {
|
||||
repo.CloneURL = repo.SSHURL
|
||||
}
|
||||
|
||||
// Fetch current project
|
||||
project, err := client.Project(strconv.Itoa(item.Id))
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
|
||||
// if no permissions we should skip the repository
|
||||
// entirely, since this should never happen
|
||||
if repo.Owner != user.Login && project.Permissions == nil {
|
||||
continue
|
||||
}
|
||||
|
||||
// if the user is the owner we can assume full access,
|
||||
// otherwise check for the permission items.
|
||||
if repo.Owner == user.Login {
|
||||
|
@ -103,6 +90,11 @@ func (r *Gitlab) GetRepos(user *model.User) ([]*model.Repo, error) {
|
|||
repo.Role.Write = true
|
||||
repo.Role.Read = true
|
||||
} else {
|
||||
// Fetch current project
|
||||
project, err := client.Project(strconv.Itoa(item.Id))
|
||||
if err != nil || project.Permissions == nil {
|
||||
continue
|
||||
}
|
||||
repo.Role.Admin = IsAdmin(project)
|
||||
repo.Role.Write = IsWrite(project)
|
||||
repo.Role.Read = IsRead(project)
|
||||
|
|
Loading…
Reference in a new issue