Merge branch 'hotfix/165' into develop

Forward port #165

Author: weierophinney

CHANGELOG.md

@@ -44,6 +44,8 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Fixed
 
+- [#165](https://github.com/zendframework/zend-http/pull/165) fixes detection of the base URL when operating under a CLI environment.
+
 - [#149](https://github.com/zendframework/zend-http/pull/149) provides fixes to `Client::setUri()` to ensure its status as a relative
   or absolute URI is correctly memoized.
 

src/PhpEnvironment/Request.php

@@ -489,6 +489,11 @@ protected function detectBaseUrl()
             // Backtrack up the SCRIPT_FILENAME to find the portion
             // matching PHP_SELF.
 
+            $argv = $this->getServer()->get('argv', []);
+            if (isset($argv[0]) && strpos($filename, $argv[0]) === 0) {
+                $filename = substr($filename, strlen($argv[0]));
+            }
+
             $baseUrl  = '/';
             $basename = basename($filename);
             if ($basename) {

test/PhpEnvironment/RequestTest.php

@@ -791,4 +791,18 @@ public function testDetectBaseUrlDoesNotRaiseErrorOnEmptyBaseUrl()
         // If no baseUrl is detected at all, an empty string is returned.
         $this->assertEquals('', $url);
     }
+
+    public function testDetectCorrectBaseUrlForConsoleRequests()
+    {
+        $_SERVER['argv']            = ['/home/user/package/vendor/bin/phpunit'];
+        $_SERVER['argc']            = 1;
+        $_SERVER['SCRIPT_FILENAME'] = '/home/user/package/vendor/bin/phpunit';
+        $_SERVER['SCRIPT_NAME']     = '/home/user/package/vendor/bin/phpunit';
+        $_SERVER['PHP_SELF']        = '/home/user/package/vendor/bin/phpunit';
+
+        $request = new Request();
+        $request->setRequestUri('/path/query/phpunit');
+
+        $this->assertSame('', $request->getBaseUrl());
+    }
 }