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

Reset passwords via email #3

Closed
amalloy opened this issue Apr 18, 2011 · 8 comments
Closed

Reset passwords via email #3

amalloy opened this issue Apr 18, 2011 · 8 comments
Assignees
Milestone

Comments

@amalloy
Copy link
Member

amalloy commented Apr 18, 2011

Migrating from email to github issues - might as well try out their new feature.

Original email from Dave:

One feature we really need to add to 4clojure is the ability to reset user passwords via email, and change them once a user has logged in. I should have done this earlier, but I didn't think the site would start gaining users so fast. I'll have to set up an SMTP server on EC2 as well.

@amalloy
Copy link
Member Author

amalloy commented Apr 18, 2011

Seems like this has three parts:

  1. Allow users to change passwords at any time (enter old password, then new one twice, etc)
  2. Provide a mechanism to reset password to something random
  3. Add a page with a form for email address, that resets the password for the account associated with that address, then mails out the new password.

It sounds like (3) requires some work from @dbyrne, to get an SMTP server going etc, but the other two sound like something @gigasquid or I could do. Interested, Carin?

@gigasquid
Copy link
Member

Yes I am interested.

@amalloy amalloy reopened this Apr 20, 2011
@ghost ghost assigned gigasquid Apr 20, 2011
@amalloy
Copy link
Member Author

amalloy commented Apr 20, 2011

The comment&close button strikes again! They should paint that thing bright red.

Anyway, glad you're interested. I'd probably start with adding a way for a user to set a new password. I'm not sure how much guidance/advice you're looking for, being new to Clojure but not to programming; I'm in IRC most of the time, so ping me if you need something.

amalloy added a commit that referenced this issue Apr 22, 2011
-  @dbyrne to write the send-email function
-  someone (@cmeier?) to add links to /login/reset and /login/update somewhere appropriate
@amalloy
Copy link
Member Author

amalloy commented Apr 22, 2011

Not sure if a commit message triggers notifications, so: this issue needs attention from @gigasquid and either @Raynes or @dbyrne. See above commit message for clarification.

@Raynes
Copy link
Contributor

Raynes commented Apr 22, 2011

I'll write send-email tomorrow morning. I've done it before.

@gigasquid gigasquid reopened this Apr 22, 2011
@gigasquid
Copy link
Member

The comment and close button is my nemesis. I am working on the frontend links.

@amalloy
Copy link
Member Author

amalloy commented Apr 22, 2011

Awesome, guys! Hopefully we can get 0.1.0 out today, it sounds like!

@amalloy
Copy link
Member Author

amalloy commented Apr 22, 2011

What a team effort this was. Issue closed, 0.1.0 deployed. Good work, everyone.

@amalloy amalloy closed this as completed Apr 22, 2011
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

3 participants