Homec4science

Application Emails - make various user email editing paths respect application…

Authored by Bob Trahan <btrahan@phacility.com> on Jan 29 2015, 23:41.

Description

Application Emails - make various user email editing paths respect application emails

Summary: Ref T3404. The only mildly sketchy bit is these codepaths all load the application email directly, by-passing privacy. I think this is necessary because not getting to see an application doesn't mean you should be able to break the application by registering a colliding email address.

Test Plan:
Tried to add a registered application email to a user account via the web ui and got a pretty error.
Ran unit tests.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T3404

Differential Revision: https://secure.phabricator.com/D11565

Details

Committed
Bob Trahan <btrahan@phacility.com>Jan 29 2015, 23:41
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1077e7a80c44: Application Emails - conditionally pass around the application email
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH7d309a8e46ca: Application Emails - make various user email editing paths respect application… (authored by Bob Trahan <btrahan@phacility.com>).Jan 29 2015, 23:41