You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Igor Bondarenko <je...@gmail.com> on 2015/04/09 14:09:46 UTC

[allura:tickets] Re: #7826 Validate email format on /auth/forgotten_password

- Fork the main repo by pressing "Fork" button in the left sidebar at https://forge-allura.apache.org/p/allura/git
- `git push` your changes to your fork
- Create a merge request by pressing "Request Merge" (left sidebar on your fork's main page)
- Post a link to your merge request here, someone from the core team will review it


---

** [tickets:#7826] Validate email format on /auth/forgotten_password**

**Status:** open
**Milestone:** unreleased
**Labels:** ux 
**Created:** Wed Feb 04, 2015 06:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Apr 09, 2015 07:11 AM UTC
**Owner:** nobody

The form at /auth/forgotten_password should validate the email address format.  Currently you could put "afd3wrfes" as your email address and Allura will blindly go ahead and say "If the given email address is on record, a password reset email has been sent to the account's primary email address." even though obviously the email isn't even possible to be valid.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] Re: #7826 Validate email format on /auth/forgotten_password

Posted by Dave Brondsema <da...@brondsema.net>.
Thanks Pranav!  Great to see a new contribution like this.  I've posted my feedback about the code changes on the merge request page.


---

** [tickets:#7826] Validate email format on /auth/forgotten_password**

**Status:** open
**Milestone:** unreleased
**Labels:** ux 
**Created:** Wed Feb 04, 2015 06:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Apr 09, 2015 07:11 AM UTC
**Owner:** nobody

The form at /auth/forgotten_password should validate the email address format.  Currently you could put "afd3wrfes" as your email address and Allura will blindly go ahead and say "If the given email address is on record, a password reset email has been sent to the account's primary email address." even though obviously the email isn't even possible to be valid.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] Re: #7826 Validate email format on /auth/forgotten_password

Posted by Pranav Sharma <pr...@gmail.com>.
When I Forked the project,it gave me the 42cc_1532 branch. In that branch, the file I want to edit, is totally in new format. How may I resolve this?


---

** [tickets:#7826] Validate email format on /auth/forgotten_password**

**Status:** open
**Milestone:** unreleased
**Labels:** ux 
**Created:** Wed Feb 04, 2015 06:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Apr 09, 2015 07:11 AM UTC
**Owner:** nobody

The form at /auth/forgotten_password should validate the email address format.  Currently you could put "afd3wrfes" as your email address and Allura will blindly go ahead and say "If the given email address is on record, a password reset email has been sent to the account's primary email address." even though obviously the email isn't even possible to be valid.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.