Convert email address obtained from LDAP to lowercase before comparin…

wekan / ldap

LDAP support for Wekan code has been moved to https://github.com/wekan/wekan/tree/master/packages/wekan-ldap , issues to https://github.com/wekan/wekan/issues , and if PRs are needed please add them instead to https://github.com/wekan/wekan/pulls

https://github.com/wekan/wekan/tree/master/packages/wekan-ldap

MIT License

12 stars 10 forks source link

Convert email address obtained from LDAP to lowercase before comparin… #82

Closed njpaul closed 3 weeks ago

njpaul commented 4 years ago

Convert email address obtained from LDAP to lowercase before comparing it for merging with an existing account.
Fix a log message which discarded the second part of the message.

Resolves #81

ghost commented 4 years ago

DeepCode's analysis on #68a9b0 found:

2 minor issues.

👉 View analysis in DeepCode’s Dashboard

:relaxed: If you want to provide feedback on our bot, here is how to contact us.

xet7 commented 3 weeks ago

Well, this is merged only to old LDAP code. Newest LDAP code is here, and may have other newer changes

https://github.com/wekan/wekan/blob/main/packages/wekan-ldap/server/loginHandler.js

https://github.com/wekan/wekan/blob/main/packages/wekan-ldap/server/sync.js

I don't have LDAP server, so I can not test.