From c3543c80cd6bfeca9f3ebcb071168869989a9574 Mon Sep 17 00:00:00 2001 From: Nick Harper Date: Wed, 1 Feb 2012 22:11:49 -0800 Subject: [PATCH] Fix conduit host check Summary: conduit was using getProductionURI instead of getURI for checking that the request was sent to the correct host, which causes problems in some dev environments Test Plan: echo '{}' | arc call-conduit --conduit-uri=mydevserver where my dev server is configured with phabricator.production-uri pointing to prod instead of my devserver Reviewers: epriestley, btrahan, jungejason Reviewed By: epriestley CC: aran, epriestley Differential Revision: https://secure.phabricator.com/D1543 --- src/applications/conduit/method/base/ConduitAPIMethod.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/applications/conduit/method/base/ConduitAPIMethod.php b/src/applications/conduit/method/base/ConduitAPIMethod.php index 3ba1a83760..5fe2e93faa 100644 --- a/src/applications/conduit/method/base/ConduitAPIMethod.php +++ b/src/applications/conduit/method/base/ConduitAPIMethod.php @@ -1,7 +1,7 @@ setPath('/'); $host = (string)$host; - $self = PhabricatorEnv::getProductionURI('/'); + $self = PhabricatorEnv::getURI('/'); if ($self !== $host) { throw new Exception( "Your client is connecting to this install as '{$host}', but it is ".