-
Notifications
You must be signed in to change notification settings - Fork 37
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
signup page forward to not found page after signup and logout #46
Comments
Thanks for the feedback! @nak3 This is actually an intended behavior. The signup page won't work if there is already a registered user. The related code can be found here: app/handler/auth.go#L34 The reason behind this is that we don't want other people to sign up the site and have control of the content. Maybe we can add a feature in the future to allow the admin user to send invitation link to other people for them to sign up and add content to the site, and add a role-based permission system(e.g. a user can be |
Hi @dinever Thank you for your explanation. I understand it. So, we should have one feature and one fix. In terms of feature, as you explained, it will be nice to have the feature to send an invitation by admin user. Otherwise I feel we can have admin page which have a button to Then, as for a fix, |
In any case, please feel free to close this report, if you would like. Or if you would like to track the feature/fix, I'm alright to leave this open. |
I also reproduced this issue and saw there was an 500 internal server error:
I agree with @nak3 , if there is already a registered user, we'd better hide that "sign up" button. |
… if there is already a registered user.
… if there is already a registered user.
Please see reproduce steps:
step-1. current version
step-2. Start dingo
step-3. Access and signup Dingo
http://localhost:8000/signup/
step-4. Logout
step-5. Go to sign up page
It returns not found page. (Even from http://localhost:8000/login to push SIGN UP, it shows not found page.)
The text was updated successfully, but these errors were encountered: