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

Improve workflow for missing certificates.

Browse source
This commit is contained in:
epriestley 2011-04-10 13:09:31 -07:00
parent 0472a5ff95
commit d87fda8561
1 changed files with 19 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
scripts/arcanist.php
View file

@ -190,17 +190,28 @@ try {
)); ));
} catch (ConduitClientException $ex) { } catch (ConduitClientException $ex) {
if ($ex->getErrorCode() == 'ERR-NO-CERTIFICATE') { if ($ex->getErrorCode() == 'ERR-NO-CERTIFICATE') {
$no_cert_msg = "You don't have certificate for ".$conduit_uri.".\n"; $message =
"\n\n".
phutil_console_format(
"YOU NEED TO __INSTALL A CERTIFICATE__ TO LOGIN TO PHABRICATOR").
"\n\n".
"The server '{$conduit_uri}' rejected your request:".
"\n\n".
$ex->getMessage().
"\n\n";
$hosts_with_cert = ifilter($hosts_config, 'cert'); $hosts_with_cert = ifilter($hosts_config, 'cert');
if (!empty($hosts_with_cert)) { if (!empty($hosts_with_cert)) {
$no_cert_msg .= 'You have certificate(s) for '. if (count($hosts_with_cert) == 1) {
implode(array_keys($hosts_with_cert), ', ').".\n"; $message .=
"You currently have a certificate installed for one host:\n\n";
} else {
$message .=
"You currently have certificates installed for these hosts:\n\n";
}
$message .= ' '.implode("\n ", array_keys($hosts_with_cert));
$message .= "\n";
} }
$no_cert_msg .= 'Please refer to page http://www.phabricator.com'. throw new ArcanistUsageException($message);
'/docs/phabricator/article/Installing_Arcanist_Certificates.html '.
'for more info.';
throw new ArcanistUsageException($no_cert_msg);
} else { } else {
throw $ex; throw $ex;
} }