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

User self registration doesn't validate email address #32

Open
tiran opened this issue Aug 20, 2015 · 2 comments
Open

User self registration doesn't validate email address #32

tiran opened this issue Aug 20, 2015 · 2 comments

Comments

@tiran
Copy link
Member

tiran commented Aug 20, 2015

The self registration workflow doesn't validate the user's email address yet. It's common practice to validate the email address.

Suggested workflow

  1. User registers herself with username, temporary password and email address. The stage user is not created. Instead the information are stored in a local sqlite database.
  2. Portal sends user an email with a validation token and a temporary password
  3. User clicks on link and validates the token.
  4. Portal creates stage user in FreeIPA.

Security concern

The temporary password is send plain text. Since it's a temporary password (user has to set a new password in FreeIPA web UI anyway) it is not so bad.

In order to mitigate flooding and DoS attacks on FreeIPA and LDAP servers, the portal must create the staging user after the mail address has been validated.

@frasertweedale
Copy link
Contributor

I agree with deferring password prompt until it is needed (step 3).

@tiran
Copy link
Member Author

tiran commented Aug 24, 2015

@simo5 has pointed out that the portal could be abused to flood the database with users. The system should mitigate flooding attacks.

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

2 participants