Closed Bug 1096327 Opened 10 years ago Closed 10 years ago

Invalid e-mail format is accepted in FxA-migration window at “Sign In”

Categories

(Marketplace Graveyard :: Consumer Pages, defect, P3)

Avenir
x86_64
Windows 7
defect

Tracking

(Not tracked)

VERIFIED FIXED
2014-11-11

People

(Reporter: julian.mozbugs, Unassigned)

References

Details

(Keywords: regression)

Steps to reproduce: 1. Load MP-stage homepage ( https://marketplace.allizom.org/ ) 2. Click the “Sign in” button from the page header. 3. Enter an invalid e-mail ( i.e. test@t) Expected results: The message “Valid e-mail required” is displayed. The "Next" button is gray and inactive. Actual results: The “Next” button is blue and active. The user can continue with the invalid entered email. Verified on FF36 (Win 7). Issue is also reproducing on MP-dev. Screencast for this issue: http://screencast.com/t/ZxhW3IOEZ9kh
Priority: -- → P3
foo@tld (with no dots) is actually a valid email address: "In the case of a top-level domain used by itself in an email address, a single string is used without any dots" [1] Which is probably why the email type is quite liberal in what it accepts. [1] http://tools.ietf.org/html/rfc5321#section-2.3.5
Based on comment 1.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
If the entered email is valid, then the "Valid email required" error should not be displayed in the registration form. Please see screencast http://screencast.com/t/N9XYARdB
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
(In reply to Iulian Timis from comment #3) > If the entered email is valid, then the "Valid email required" error should > not be displayed in the registration form. Please see screencast > http://screencast.com/t/N9XYARdB Technically as per comment 1 foo@tld should be allowed with no dots. But looking at fxa content server they're explicitly doing their own validation and explicitly making it look for at least one dot. https://github.com/mozilla/fxa-content-server/blob/master/app/scripts/lib/validate.js I'll raise a github issue and see what they say - if they have good reasons for doing this then we should update and follow suite even if it goes against the spec.
Blocks: 1088604
No longer blocks: 1007956
PR on the FxA side is here: https://github.com/mozilla/fxa-content-server/pull/1909/files Shane expect it'll be another couple of weeks before it sees prod.
The FxA fix for this has landed here https://github.com/mozilla/fxa-content-server/commit/5ae2aef1c5a80017e5e1e51e3e91c2ab5ac5d52b As noted in comment 6 it might be a while before this is released to prod, but there's nothing to do at our end.
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
Verified as fixed in https://marketplace.allizom.org/ on FF37 (Win 7). Postfix screencast http://screencast.com/t/QHabWyPWA7L Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.