Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

User's email control inconsistency #10723

Closed
3 tasks done
Pachat opened this issue Dec 28, 2021 · 0 comments
Closed
3 tasks done

User's email control inconsistency #10723

Pachat opened this issue Dec 28, 2021 · 0 comments

Comments

@Pachat
Copy link

Pachat commented Dec 28, 2021

Preflight Checklist

Describe the Bug

There are at least 3 places where a user's email is concerned

  • installation
  • login
  • user's detail

The login requires a valid email, but the installation and the user's detail don't.

As a result, in case of a mistake when editing the address, it is not possible to login.
Blocking the whole installation in case of the admin.

Even if this will not prevent from entering a wrong email address, it would be nice to have a control on a valid format address when installing or modifying a user's address.
Another solution would be to consider the login address as a mere username and get rid of the valid email requirement. Not to say that the email recover will then not be available.

To Reproduce

On

  • install, answer "myemail" (without @mydomain.com)
    or on
  • user's detail: modify the email to "myemail" (without @mydomain.com)

Errors Shown

The user cannot login anymore.
And if is the admin (sure admin should pay attention ;), there is no way to recover by modifying the row (as it is fortunately protected as read-only in the database).

What version of Directus are you using?

9.4.0

What version of Node.js are you using?

14.17.1

What database are you using?

SQLite

What browser are you using?

Firefox

What operating system are you using?

Windows_NT 10.0.19043

How are you deploying Directus?

locally

@directus directus locked and limited conversation to collaborators Dec 28, 2021
@rijkvanzanten rijkvanzanten converted this issue into discussion #10738 Dec 28, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant