Enforce presence of an email address in the system

RESOLVED FIXED in 2.0

Status

Cloud Services
Server: Sync
--
enhancement
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: telliott, Assigned: telliott)

Tracking

unspecified
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [AA-AT-02F])

(Assignee)

Description

8 years ago
The Weave Server's registration component contains the following code within mysql.php line 99: function create_user($password, $email = "")
The entire function is written to allow users to be created without a corresponding e-mail address. There are many areas of the application that won't operate correctly without an e-mail address. Therefore, either these areas should be modified to operate in lieu of an e-mail address, or it should be required data in order to call this function.
(Assignee)

Comment 1

8 years ago
Now enforcing email.

http://hg.mozilla.org/labs/weaveserver-registration/rev/4af2c123877b

Les, can you review, please? Note that we're not doing any form of email validation, since it's trivial for someone to get around it. If we want email verification in accounts, we'll need to open it as a new feature.
Blocks: 546750
Whiteboard: [AA-AT-02F]
Looks good to me.
(Assignee)

Updated

8 years ago
Severity: normal → enhancement
Target Milestone: --- → 2.0
(Assignee)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.