summaryrefslogtreecommitdiffstats
path: root/Dockerfile
diff options
context:
space:
mode:
authorSamuel <faust64@gmail.com>2019-04-26 18:21:42 +0200
committerGitHub <noreply@github.com>2019-04-26 18:21:42 +0200
commitb17359ec6fd0e8526a623691f87f55158999b43a (patch)
tree5bfed4144d060cc51aa2119f7dfc966576652131 /Dockerfile
parent11a91bfc786789df24a9e1d9a4b10365bfbd0deb (diff)
downloadwekan-b17359ec6fd0e8526a623691f87f55158999b43a.tar.gz
wekan-b17359ec6fd0e8526a623691f87f55158999b43a.tar.bz2
wekan-b17359ec6fd0e8526a623691f87f55158999b43a.zip
fix(oidc): can not log in
Trying to configure wekan authenticating against LemonLDAP-NG, I used to read about errors like the following: ``` XXX: getUserInfo response: { sub: 'demoone' } XXX: userinfo: { sub: 'demoone' } {"line":"431","file":"oauth.js","message":"Error in OAuth Server: id is not defined","time":{"$date":1556286530412},"level":"warn"} Exception while invoking method 'login' { stack: 'ReferenceError: id is not defined\n at Object.handleOauthRequest (packages/wekan-oidc.js:39:68)\n at OAuth._requestHandlers.(anonymous function) (packages/oauth2.js:27:31)\n at middleware (packages/oauth.js:203:5)\n at packages/oauth.js:176:5', source: 'method' } ``` Looking at the sources, that error message seems to be right: we have several references to `id`, `uid`, `displayName` or `email`, which are not defined. Probably a typo, assuming we meant these to be strings. Applying that patch, I confirm I can finally log in: ``` XXX: getUserInfo response: { sub: 'demoone' } XXX: userinfo: { sub: 'demoone' } XXX: serviceData: { id: undefined, username: undefined, fullname: undefined, accessToken: 'e57dc4e9e81cc98c279db3ed08b1c72f', expiresAt: 1556298699213, email: undefined } XXX: profile: { name: undefined, email: undefined } ``` All the credit goes to @pcurie .
Diffstat (limited to 'Dockerfile')
0 files changed, 0 insertions, 0 deletions