-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[PLT-7778] Pre-populate token/API endpoints for GitLab #130
Conversation
ddf69dc
to
edfab41
Compare
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.
Great work!
Just two minor notes/comments:
1 - On a server with Enterprise license installed, the GitLab Site URL is non-editable. Repro steps:
- upload an E20 Enterprise license on a Mattermost install
- Go to System Console > OAuth 2.0
- Select GitLab from the dropdown
- Try to edit GitLab site URL field
- Observed: Field not editable
PS: Given this is only on an Enterprise-licensed server, let me know if you have any trouble reproducing it.
2 - When populating the token/API endpoints, would it be possible to remove a trailing slash from the GitLab Site URL if one is present? I can see admins doing that by accident, resulting in a double slash for the token/API endpoints
PS: We automatically trim trailing slash from the System Console > Configuration > SiteURL setting. So that might be a good comparison.
@jasonblais Will look into it! |
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.
Awesome. Both issues fixed, verified that GitLab SSO works as before both in TE and EE.
Nice work!
Nice work @R-Wang97 ! There's an issue with our build system but as soon as that's resolved we'll merge this in |
Summary
Rather than having a user manually enter the
Token
andAPI
endpoints for GitLab, they now just need to enter the site URL, and theToken
,API
, andAuth
endpoints will be automatically generated.Change has been made for both regular and enterprise versions.
Ticket Link
mattermost/mattermost#7558
https://mattermost.atlassian.net/browse/PLT-7778
Checklist