Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
s-heinrich opened this issue Dec 7, 2017 · 8 comments
Open

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

s-heinrich opened this issue Dec 7, 2017 · 8 comments

Comments

@s-heinrich
Copy link

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.

@neurolabs
Copy link

I added a pull request that fixes this #2263

@neurolabs
Copy link

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

@xet7
Copy link
Member

xet7 commented Mar 16, 2019

@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

@neurolabs
Copy link

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

@neurolabs
Copy link

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
Copy link

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.

@AuspeXeu
Copy link

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

@Rohmilchkaese
Copy link

Still exists with OAUTH2 in Version 7.27.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants