Fix bug where the colon between host and port would appear, even when port was an empty string.

2012-11-22 16:25:34 +00:00
parent 52d9317cfd
commit bfabd20ae4
1 changed files with 5 additions and 3 deletions
--- a/src/GaryJones/OAuth/Request.php
+++ b/src/GaryJones/OAuth/Request.php
@@ -288,9 +288,11 @@ public function getNormalizedHttpUrl()
        // $host = (isset($parts['host'])) ? explode(':', $parts['host'])[0] : '';
        $path = (isset($parts['path'])) ? $parts['path'] : '';

-        if (($scheme == 'https' && $port != '443')
-            || ($scheme == 'http' && $port != '80')) {
-            $host = "$host:$port";
+        if ($port) {
+            if (('https' == $scheme && $port != '443')
+                || ('http' == $scheme && $port != '80')) {
+                $host = "$host:$port";
+            }
        }
        return "$scheme://$host$path";
    }