opentochoice email sign up issues

VERIFIED FIXED

Status

Websites Graveyard
opentochoice.org
VERIFIED FIXED
8 years ago
7 years ago

People

(Reporter: Jane Finette, Unassigned)

Tracking

Details

(Whiteboard: [target: 0.75])

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
hi there 

Barbara just forwarded an email to me from a user who couldnt sign up on the website. 

Copied his email here:

I have just registered using my mail XXX.org from your page http://opentochoice.org/es/ without problems, but a first attempt failed using XXX.info (suggesting that e-mail addresses ending in 4 letters are not accepted), so an urgent note to your webmaster is advisable.

Can we check there are no issues please? There have also been reports via Asa that others have also been having some probs. (Asa could not replicate the issues). 

thanks 
jane
I confirm the issue with .info domains
Created attachment 428955 [details] [diff] [review]
patch

Here is a patch fixing this issue by replacing the faulty regex by PHP's built-in email filtering function
OS: Mac OS X → All
Hardware: x86 → All

Comment 3

8 years ago
Fixed in r63298 please review
(In reply to comment #3)
> Fixed in r63298 please review

I got: 

"sign up confirmed

Thank you for signing up, we will send you more information when it's your time to choose and send you other news from Mozilla."

when using stephend@food.info, but I of course don't have a .info account/address with which to truly test.
Seems like the original issue was that tld's with 4 letters or more weren't recognized as valid. It doesn't send anything yet, so the fix should be OK.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [target: 0.75]
Verified FIXED on staging; see comment 4.  I checked the admin panel, too, and my test address showed up there.
Status: RESOLVED → VERIFIED

Updated

7 years ago
Component: opentochoice.org → opentochoice.org
Product: Websites → Websites Graveyard
You need to log in before you can comment on or make changes to this bug.