revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2025-03-10 11:24:48 +01:00
Code Issues Releases Wiki Activity

Fix for arc install-certificate timeout.

Browse source 
Summary:
Extended the install-certificate workflow's timeout from 5
seconds to rely on the ConduitClient default (30 seconds), which matches the HTTP interface.

Test Plan: Run arc install-certificate

Reviewers: epriestley

CC: epriestley, aran

Differential Revision: https://secure.phabricator.com/D1404
This commit is contained in:
Marke Hallowell 2012-01-14 19:55:20 -08:00 • committed by epriestley
parent e5bda05200
commit 7e63e232ba
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/workflow/install-certificate/ArcanistInstallCertificateWorkflow.php
View file

@ -1,7 +1,7 @@
<?php <?php
/* /*
* Copyright 2011 Facebook, Inc. * Copyright 2012 Facebook, Inc.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -65,7 +65,6 @@ EOTEXT
echo "Trying to connect to server...\n"; echo "Trying to connect to server...\n";
$conduit = new ConduitClient($uri); $conduit = new ConduitClient($uri);
$conduit->setTimeout(5);
try { try {
$conduit->callMethodSynchronous('conduit.ping', array()); $conduit->callMethodSynchronous('conduit.ping', array());
} catch (Exception $ex) { } catch (Exception $ex) {