Merge pull request #447 from Miouge1/master

Use read_user as default scope for GitLab
This commit is contained in:
Jehiah Czebotar 2017-09-13 10:27:36 -04:00 committed by GitHub
commit 7b26256df6
2 changed files with 2 additions and 2 deletions

View File

@ -36,7 +36,7 @@ func NewGitLabProvider(p *ProviderData) *GitLabProvider {
} }
} }
if p.Scope == "" { if p.Scope == "" {
p.Scope = "api" p.Scope = "read_user"
} }
return &GitLabProvider{ProviderData: p} return &GitLabProvider{ProviderData: p}
} }

View File

@ -53,7 +53,7 @@ func TestGitLabProviderDefaults(t *testing.T) {
p.Data().RedeemURL.String()) p.Data().RedeemURL.String())
assert.Equal(t, "https://gitlab.com/api/v3/user", assert.Equal(t, "https://gitlab.com/api/v3/user",
p.Data().ValidateURL.String()) p.Data().ValidateURL.String())
assert.Equal(t, "api", p.Data().Scope) assert.Equal(t, "read_user", p.Data().Scope)
} }
func TestGitLabProviderOverrides(t *testing.T) { func TestGitLabProviderOverrides(t *testing.T) {