Skip to content
Snippets Groups Projects

Update github.md

Merged username-removed-524699 requested to merge ffittschen/gitlab-ce:patch-1 into master

Hi, I ran into a few pitfalls while I enabled OmniAuth to be able to migrate repos from github to my omnibus-gitlab instance today, so here are my findings:

These docs state thate the default authorization callback URL is '${YOUR_DOMAIN}/import/github/callback', but the screenshot only shows '${YOUR_DOMAIN}', which works, so I removed the ambiguity.

When editing gitlab_rails['omniauth_providers'] for github.com, you have to omit the "url" => "https://github.com/", part, otherwise it won't work (OmniAuth will stay deactivated). This is backed by the configuration details of the "from-source-installation" which also omits the url in the yml file.

After changing the gitlab.rb file, sudo gitlab-ctl reconfigure needs to be run, which is missing in the docs.

Following the updated version of this documentation I was able to enable OmniAuth.

If anything I edited is wrong, please correct me, as I don't know how the "from-source-installations" would react to my changes.

Best regards, Florian

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading