Merge pull request #31 from 99designs/get-request-target

SigningString: use getRequestTarget not getPath & getQuery.
99designs · May 4, 2017 · 2d45a93 · 2d45a93
2 parents fd2d274 + 6d777cb
commit 2d45a93
Showing 1 changed file with 1 addition and 15 deletions.
diff --git a/src/SigningString.php b/src/SigningString.php
@@ -78,21 +78,7 @@ private function requestTargetLine()
         return sprintf(
             '(request-target): %s %s',
             strtolower($this->message->getMethod()),
-            $this->getPathWithQueryString()
+            $this->message->getRequestTarget()
         );
     }
-
-    /**
-     * @return string
-     */
-    private function getPathWithQueryString()
-    {
-        $path = $this->message->getUri()->getPath();
-        $qs = $this->message->getUri()->getQuery();
-        if (empty($qs)) {
-            return $path;
-        } else {
-            return "$path?$qs";
-        }
-    }
 }