revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-20 05:42:40 +01:00
Code Issues Releases Wiki Activity

OAuth - make sure users know they are exposing their primary email address

Browse source 
Summary: Fixes T7263. Last bit there was to upgrade this dialogue to let users know they are letting their primary email address be exposed in these flows. Depends on D11791, D11792, at least in terms of being accurate to the user as the code ended up strangely decoupled.

Test Plan: wordsmithin'

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T7263

Differential Revision: https://secure.phabricator.com/D11793
This commit is contained in:
Bob Trahan 2015-02-17 14:19:33 -08:00
parent d6bbbcb620
commit 17ced84ace
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/applications/oauthserver/controller/PhabricatorOAuthServerAuthController.php
View file

@ -244,7 +244,8 @@ final class PhabricatorOAuthServerAuthController
->appendParagraph( ->appendParagraph(
pht( pht(
'Do you want to authorize the external application "%s" to '. 'Do you want to authorize the external application "%s" to '.
'access your Phabricator account data?', 'access your Phabricator account data, including your primary '.
'email address?',
phutil_tag('strong', array(), $name))) phutil_tag('strong', array(), $name)))
->appendChild($form->buildLayoutView()) ->appendChild($form->buildLayoutView())
->addSubmitButton(pht('Authorize Access')) ->addSubmitButton(pht('Authorize Access'))