wekan icon indicating copy to clipboard operation
wekan copied to clipboard

The invitation code doesn't exist - case-sensitive eMail

Open s-heinrich opened this issue 7 years ago • 8 comments

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.

s-heinrich avatar Dec 07 '17 12:12 s-heinrich

I added a pull request that fixes this #2263

neurolabs avatar Mar 16 '19 13:03 neurolabs

Can this be tested on any testing installation? I would be happy to do so if you point me towards it.

neurolabs avatar Mar 16 '19 15:03 neurolabs

@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

xet7 avatar Mar 16 '19 17:03 xet7

My change somehow does not fix this issue. Will check further.

neurolabs avatar Mar 18 '19 09:03 neurolabs

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()

neurolabs avatar Mar 18 '19 12:03 neurolabs

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.

neurolabs avatar Apr 05 '19 10:04 neurolabs

Same issue still persists for me on docker image with tag quay.io/wekan/wekan:v4.16

AuspeXeu avatar Jun 24 '20 15:06 AuspeXeu

Still exists with OAUTH2 in Version 7.27.0

Rohmilchkaese avatar Jan 03 '24 15:01 Rohmilchkaese