Skip to content

Commit b749d50

Browse files
authored
Merge pull request #1082 from cloudyu/pull
Fix wrong config options In `./lib/web/auth/` some config includes still used `config.serverurl` instead of the correct `config.serverURL`. This causes wrong URL in worst case. This patch should fix those problems and migrate the wrong statements to camelcase.
2 parents 769a1c4 + 35a9f72 commit b749d50

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

lib/web/auth/oauth2/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ oauth2Auth.get('/auth/oauth2', function (req, res, next) {
100100
// github auth callback
101101
oauth2Auth.get('/auth/oauth2/callback',
102102
passport.authenticate('oauth2', {
103-
successReturnToOrRedirect: config.serverurl + '/',
104-
failureRedirect: config.serverurl + '/'
103+
successReturnToOrRedirect: config.serverURL + '/',
104+
failureRedirect: config.serverURL + '/'
105105
})
106106
)

lib/web/auth/openid/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ openIDAuth.post('/auth/openid', urlencodedParser, function (req, res, next) {
5555
// openID auth callback
5656
openIDAuth.get('/auth/openid/callback',
5757
passport.authenticate('openid', {
58-
successReturnToOrRedirect: config.serverurl + '/',
59-
failureRedirect: config.serverurl + '/'
58+
successReturnToOrRedirect: config.serverURL + '/',
59+
failureRedirect: config.serverURL + '/'
6060
})
6161
)

0 commit comments

Comments
 (0)