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

Support self service registration #340

Closed
kaiyou opened this issue Dec 2, 2017 · 10 comments
Closed

Support self service registration #340

kaiyou opened this issue Dec 2, 2017 · 10 comments
Milestone

Comments

@kaiyou
Copy link
Member

kaiyou commented Dec 2, 2017

Currently Mailu only supports account creation by a domain manager. It would be interesting to add two new features:

  • account self-service registration
  • domain self-service registration

Regarding account registration, an additional field in domains, that can be set by a global administrator, will allow for registration on the domain. Registration will still honor user count limits if set.

Domain registration will first require that the domain owner add a TXT entry to the zone including the Mailu instance main hostname, thus confirming the requester is legitimate for registering the domain. Then an initial account will be created, that is domain manager for the domain.

@kaiyou kaiyou added the feature label Dec 2, 2017
@kaiyou kaiyou added this to the 1.6 milestone Dec 2, 2017
@kaiyou
Copy link
Member Author

kaiyou commented Dec 3, 2017

A first version of self-service user signup is enabled on master, without any captcha for now. User signup can be enabled for a domain in the domain config.

@kaiyou
Copy link
Member Author

kaiyou commented Dec 3, 2017

This lib was suggested by SaiGonSeamus: https://github.com/lepture/captcha

@kaiyou
Copy link
Member Author

kaiyou commented Dec 3, 2017

Also, I made sure that chosing the domain is a first step process and the domain is specified in the query string, so that one can link to the registration form for a specific domain.

@kaiyou
Copy link
Member Author

kaiyou commented Dec 3, 2017

We'll also have to be a bit more careful when checking the localpart format before releasing this

@kaiyou
Copy link
Member Author

kaiyou commented Dec 3, 2017

Unffortunately, I did not find any proper captcha library for Python flask that can use a Redis backend for storing captchas.

@whitef0x0
Copy link

@kaiyou is this on hold or permanently closed?

@kaiyou
Copy link
Member Author

kaiyou commented Dec 18, 2017

It is actually implemented and missing a captcha before we can merge it.

@kaiyou
Copy link
Member Author

kaiyou commented Dec 18, 2017

So it should be available in the next release

@kaiyou
Copy link
Member Author

kaiyou commented Apr 11, 2018

After some long hours trying to get something decent working, I finally decided to go with an optional reCaptcha. I added a warning about the obvious privacy issues.

@kaiyou
Copy link
Member Author

kaiyou commented Apr 21, 2018

This is now implemented in master.

@kaiyou kaiyou closed this as completed Apr 21, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants