wekan
wekan copied to clipboard
The invitation code doesn't exist - case-sensitive eMail
Issue
Impacted version: 0.60
Server Setup Information:
- Docker-Image (wekanteam/wekan:latest, 0.60)
- Operating System: CentOS 7.4
- Deployment Method(tar/sandstorm/snap/docker/etc /http frontend (nginx, httpd, ...)):
- Docker: wekanteam/wekan:latest, 0.60
- MongoDB Version: 3.2.18
- ROOT_URL environment variable (Is there a subfolder?): /wekan
Problem description: Wekan with disable self-regitration: if i invite a new user, i can write the eMail non-case sensitiv. In the login-dialog, the eMail will be autom. change to lower-case. The user can't login with the error message "The invitation code doesn't exist" if the email-adress in the database with uppercase letters.
I added a pull request that fixes this #2263
Can this be tested on any testing installation? I would be happy to do so if you point me towards it.
@neurolabs
You can test it this way:
git clone https://github.com/wekan/wekan
cd wekan
edit docker-compose.yml to have this:
image: quay.io/wekan/wekan:devel
Also add email settings to docker-compose.yml.
Then start Wekan in docker:
docker-compose up -d
And then use webbrowser to go to http://localhost , register new account that will become admin.
Also see: https://github.com/wekan/wekan/wiki/Settings https://github.com/wekan/wekan/wiki/Troubleshooting-Mail https://github.com/wekan/wekan/wiki/Adding-users
My change somehow does not fix this issue. Will check further.
I finally managed to sucessfully test this in a locally built docker container. But the wekan:devel container on quay.io does not include the changes (yet). I inspected the container and it contained the old code without the adition of toLowerCase()
I can confirm that in the docker image 2.56.0, the error message "The invitation code doesn't exist" does not appear, but instead I see the error message from #2310 and the account gets created.
Same issue still persists for me on docker image with tag quay.io/wekan/wekan:v4.16
Still exists with OAUTH2 in Version 7.27.0