Changeset View
Changeset View
Standalone View
Standalone View
src/applications/auth/provider/PhabricatorAmazonAuthProvider.php
<?php | <?php | ||||
final class PhabricatorAmazonAuthProvider | final class PhabricatorAmazonAuthProvider | ||||
extends PhabricatorOAuth2AuthProvider { | extends PhabricatorOAuth2AuthProvider { | ||||
public function getProviderName() { | public function getProviderName() { | ||||
return pht('Amazon'); | return pht('Amazon'); | ||||
} | } | ||||
protected function getProviderConfigurationHelp() { | protected function getProviderConfigurationHelp() { | ||||
$login_uri = PhabricatorEnv::getURI($this->getLoginURI()); | $login_uri = PhabricatorEnv::getURI($this->getLoginURI()); | ||||
$uri = new PhutilURI(PhabricatorEnv::getProductionURI('/')); | $uri = new PhutilURI(PhabricatorEnv::getProductionURI('/')); | ||||
$https_note = null; | $https_note = null; | ||||
if ($uri->getProtocol() !== 'https') { | if ($uri->getProtocol() !== 'https') { | ||||
$https_note = pht( | $https_note = pht( | ||||
'NOTE: Amazon **requires** HTTPS, but your Phabricator install does '. | 'NOTE: Amazon **requires** HTTPS, but your installation does '. | ||||
epriestley: I think just "your install" would be reasonable here, although arguably we should stop using… | |||||
'not use HTTPS. **You will not be able to add Amazon as an '. | 'not use HTTPS. **You will not be able to add Amazon as an '. | ||||
'authentication provider until you configure HTTPS on this install**.'); | 'authentication provider until you configure HTTPS on this '. | ||||
'installation**.'); | |||||
} | } | ||||
return pht( | return pht( | ||||
"%s\n\n". | "%s\n\n". | ||||
"To configure Amazon OAuth, create a new 'API Project' here:". | "To configure Amazon OAuth, create a new 'API Project' here:". | ||||
"\n\n". | "\n\n". | ||||
"http://login.amazon.com/manageApps". | "http://login.amazon.com/manageApps". | ||||
"\n\n". | "\n\n". | ||||
Show All 19 Lines |
I think just "your install" would be reasonable here, although arguably we should stop using "install" as a noun and say "your installation" or "this installation" instead (and below, "...on this install.").