Merge pull request #51 from WyriHaximus-labs/fix-ucfirst-cname-filter-failure

Ensure we ignore the case of the answer

Author: WyriHaximus

src/Resolver/Resolver.php

@@ -85,8 +85,9 @@ private function filterByType(array $answers, $type)
 
     private function filterByField(array $answers, $field, $value)
     {
+        $value = strtolower($value);
         return array_filter($answers, function ($answer) use ($field, $value) {
-            return $value === $answer->$field;
+            return $value === strtolower($answer->$field);
         });
     }
 

tests/Resolver/ResolverTest.php

@@ -31,6 +31,27 @@ public function resolveShouldQueryARecords()
         $resolver->resolve('igor.io')->then($this->expectCallableOnceWith('178.79.169.131'));
     }
 
+    /** @test */
+    public function resolveShouldQueryARecordsAndIgnoreCase()
+    {
+        $executor = $this->createExecutorMock();
+        $executor
+            ->expects($this->once())
+            ->method('query')
+            ->with($this->anything(), $this->isInstanceOf('React\Dns\Query\Query'))
+            ->will($this->returnCallback(function ($nameserver, $query) {
+                $response = new Message();
+                $response->header->set('qr', 1);
+                $response->questions[] = new Record('Blog.wyrihaximus.net', $query->type, $query->class);
+                $response->answers[] = new Record('Blog.wyrihaximus.net', $query->type, $query->class, 3600, '178.79.169.131');
+
+                return Promise\resolve($response);
+            }));
+
+        $resolver = new Resolver('8.8.8.8:53', $executor);
+        $resolver->resolve('blog.wyrihaximus.net')->then($this->expectCallableOnceWith('178.79.169.131'));
+    }
+
     /** @test */
     public function resolveShouldFilterByName()
     {