HomePhabricator

Fix an issue with user.whoami

Description

Fix an issue with user.whoami

Summary:
Both user.whoami and user.query call the same wire formatting code, but expect different data.

Don't try to add availability data to user.whoami.

Stop adding email data to user.query. We've added it since D11791, but my intent was for it to be exposed only via user.whoami (i.e., expose your address, not others').

Test Plan:

  • Called both methods.
  • Saw emails on user.whoami.
  • Saw availability on user.query.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

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

Details

Provenance
epriestleyAuthored on
epriestleyPushed on May 14 2015, 11:48 PM
Reviewer
btrahan
Differential Revision
D12848: Fix an issue with user.whoami
Parents
rPb34d5bf44ba9: Strip GMail French quote header
Branches
Unknown
Tags
Unknown

Event Timeline

Not exposing email addresses with user.query makes it difficult to do things like associate phabricator users with slack users for bots that post phabricator notifications to slack :(