Andreas/wekan
1
0
Fork 0
mirror of https://github.com/wekan/wekan.git synced 2025-12-25 19:58:49 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 5
wekan/packages
History
Samuel b17359ec6f
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 .
2019-04-26 18:21:42 +02:00
..
kadira-flow-router Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
markdown Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
meteor-accounts-cas Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
meteor-useraccounts-core Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
wekan-accounts-cas Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
wekan-accounts-oidc Include to Wekan packages directory contents, so that meteor command would build all directly. 2019-04-20 15:18:33 +03:00
wekan-ldap Update package names. 2019-04-20 15:24:09 +03:00
wekan-oidc fix(oidc): can not log in 2019-04-26 18:21:42 +02:00
wekan-scrollbar Update package names. 2019-04-20 15:24:09 +03:00