Skip to content

Commit

Permalink
fix: github enterprise auth compatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
NGPixel committed Jul 14, 2019
1 parent d74683b commit 8b71222
Show file tree
Hide file tree
Showing 2 changed files with 33 additions and 6 deletions.
21 changes: 15 additions & 6 deletions server/modules/authentication/github/authentication.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,22 @@ const _ = require('lodash')

module.exports = {
init (passport, conf) {
let githubConfig = {
clientID: conf.clientId,
clientSecret: conf.clientSecret,
callbackURL: conf.callbackURL,
scope: ['user:email']
}

if (conf.useEnterprise) {
githubConfig.authorizationURL = `https://${conf.enterpriseDomain}/login/oauth/authorize`
githubConfig.tokenURL = `https://${conf.enterpriseDomain}/login/oauth/access_token`
githubConfig.userProfileURL = conf.enterpriseUserEndpoint
githubConfig.userEmailURL = `${conf.enterpriseUserEndpoint}/emails`
}

passport.use('github',
new GitHubStrategy({
clientID: conf.clientId,
clientSecret: conf.clientSecret,
callbackURL: conf.callbackURL,
scope: ['user:email']
}, async (accessToken, refreshToken, profile, cb) => {
new GitHubStrategy(githubConfig, async (accessToken, refreshToken, profile, cb) => {
try {
const user = await WIKI.models.users.processProfile({
profile: {
Expand Down
18 changes: 18 additions & 0 deletions server/modules/authentication/github/definition.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,22 @@ props:
title: Client Secret
hint: Application Client Secret
order: 2
useEnterprise:
type: Boolean
title: Use GitHub Enterprise
hint: Enable if you're using the self-hosted GitHub Enterprise version
default: false
order: 3
enterpriseDomain:
type: String
title: GitHub Enterprise Domain
hint: GitHub Enterprise Only - Domain of your installation (e.g. github.company.com). Leave blank otherwise.
default: ''
order: 4
enterpriseUserEndpoint:
type: String
title: GitHub Enterprise User Endpoint
hint: GitHub Enterprise Only - Endpoint to fetch user details (e.g. https://api.github.com/user). Leave blank otherwise.
default: 'https://api.github.com/user'
order: 5

0 comments on commit 8b71222

Please sign in to comment.