New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
connector: add GitLab connecor #788
Conversation
givia
commented
Jan 25, 2017
// Open returns a strategy for logging in through GitLab. | ||
func (c *Config) Open(logger logrus.FieldLogger) (connector.Connector, error) { | ||
return &gitlabConnector{ | ||
baseURL: c.BaseURL, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would you like to provide a default value for baseURL that corresponds to the public gitlab API URL? This would be helpful if the user does not provide one via the config.
name: GitLab | ||
config: | ||
# Credentials can be string literals or pulled from the environment. | ||
baseURL: https://www.gitlab.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a note that this is optional
} | ||
|
||
var gitlabGroups []gitlabGroup | ||
if err := json.NewDecoder(resp.Body).Decode(&gitlabGroups); err != nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does gitlab do pagination? ref #706
e899140
to
8e4fbab
Compare
Successfully tested https://github.com/coreos/dex/tree/master/cmd/example-app with the new gitlab connector 👍 Groups scope works as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Would be great if you could squash your commits. Thanks |
555a27c
to
e623ad4
Compare