Merge pull request #138 from dman-coders/url-parsing

More complete support of URL parsing

Author: oscarotero

src/Url.php

@@ -415,10 +415,20 @@ protected function buildUrl()
             $url .= $this->info['scheme'].'://';
         }
 
+        $user = isset($this->info['user']) ? $this->info['user'] : '';
+        $pass = isset($this->info['pass']) ? ':' . $this->info['pass']  : '';
+        if ($user || $pass) {
+            $url .= $user . $pass . '@';
+        }
+
         if (isset($this->info['host'])) {
             $url .= $this->info['host'];
         }
 
+        if (isset($this->info['port'])) {
+            $url .= ':' . $this->info['port'];
+        }
+
         $url .= $this->getPath();
 
         if (!empty($this->info['query'])) {

tests/UrlTest.php

@@ -15,6 +15,9 @@ public function urlsParserProvider()
             ['https://animoto.com/oembeds/create.xml?automated=true&options=start_hq', 'https://animoto.com/oembeds/create.xml?automated=true&options=start_hq'],
             ['http://static2.politico.com/dims4/default/28fb355/2147483647/resize/1160x%3E/quality/90/?url=http%3A%2F%2Fs3-origin-images.politico.com%2F2013%2F12%2F18%2F131218_george_w_bush_barack_obama_ap_60', 'http://static2.politico.com/dims4/default/28fb355/2147483647/resize/1160x%3E/quality/90/?url=http%3A%2F%2Fs3-origin-images.politico.com%2F2013%2F12%2F18%2F131218_george_w_bush_barack_obama_ap_60'],
             ['https://plus.google.com/+carlsenverlag/posts/2hibgWrmhp1', 'https://plus.google.com/+carlsenverlag/posts/2hibgWrmhp1'],
+            ['http://test.drupal.dd:8083/tests/localport.html', 'http://test.drupal.dd:8083/tests/localport.html'],
+            ['http://[email protected]:8083/tests/identified.html', 'http://[email protected]:8083/tests/identified.html'],
+            ['http://testuser:[email protected]:8083/tests/authenticated.html', 'http://testuser:[email protected]:8083/tests/authenticated.html'],
         ];
     }