Merge branch 'refs/heads/fix-dns-ipv6'

Author: cboden

src/React/Dns/Resolver/Factory.php

@@ -46,6 +46,15 @@ protected function createCachedExecutor(LoopInterface $loop)
 
     protected function addPortToServerIfMissing($nameserver)
     {
-        return false === strpos($nameserver, ':') ? "$nameserver:53" : $nameserver;
+        if (strpos($nameserver, '[') === false && substr_count($nameserver, ':') >= 2) {
+            // several colons, but not enclosed in square brackets => enclose IPv6 address in square brackets
+            $nameserver = '[' . $nameserver . ']';
+        }
+        // assume a dummy scheme when checking for the port, otherwise parse_url() fails
+        if (parse_url('dummy://' . $nameserver, PHP_URL_PORT) === null) {
+            $nameserver .= ':53';
+        }
+
+        return $nameserver;
     }
 }

tests/React/Tests/Dns/Resolver/FactoryTest.php

@@ -41,6 +41,33 @@ public function createCachedShouldCreateResolverWithCachedExecutor()
         $this->assertInstanceOf('React\Dns\Query\CachedExecutor', $this->getResolverPrivateMemberValue($resolver, 'executor'));
     }
 
+    /**
+     * @test
+     * @dataProvider factoryShouldAddDefaultPortProvider
+     */
+    public function factoryShouldAddDefaultPort($input, $expected)
+    {
+        $loop = $this->getMock('React\EventLoop\LoopInterface');
+
+        $factory = new Factory();
+        $resolver = $factory->create($input, $loop);
+
+        $this->assertInstanceOf('React\Dns\Resolver\Resolver', $resolver);
+        $this->assertSame($expected, $this->getResolverPrivateMemberValue($resolver, 'nameserver'));
+    }
+
+    public static function factoryShouldAddDefaultPortProvider()
+    {
+        return array(
+            array('8.8.8.8',        '8.8.8.8:53'),
+            array('1.2.3.4:5',      '1.2.3.4:5'),
+            array('localhost',      'localhost:53'),
+            array('localhost:1234', 'localhost:1234'),
+            array('::1',            '[::1]:53'),
+            array('[::1]:53',       '[::1]:53')
+        );
+    }
+
     private function getResolverPrivateMemberValue($resolver, $field)
     {
         $reflector = new \ReflectionProperty('React\Dns\Resolver\Resolver', $field);