Merge pull request #70 from RageZBla/67-handle-404-gracefully

Try to process the page even when the server returned 4xx or 5xxx.

Author: shweshi

src/OpenGraph.php

@@ -70,7 +70,7 @@ protected function curl_get_contents($url, $lang, $userAgent)
 
         curl_setopt_array($curl, [
             CURLOPT_URL            => $url,
-            CURLOPT_FAILONERROR    => true,
+            CURLOPT_FAILONERROR    => false,
             CURLOPT_FOLLOWLOCATION => true,
             CURLOPT_RETURNTRANSFER => true,
             CURLOPT_SSL_VERIFYHOST => false,

tests/OpenGraphTest.php

@@ -48,6 +48,17 @@ public function testFetchReturnsEmptyArrayForWebsiteWithNoMetadata()
         $this->assertEmpty($data);
     }
 
+    /** @test */
+    public function testFetchReturnsEmptyArrayForWebsiteWithNoMetadataAndReturnedNotFound()
+    {
+        $opengraph = new OpenGraph();
+        $data = $opengraph->fetch(
+            'https://www.example.com/not-found'
+        );
+
+        $this->assertEmpty($data);
+    }
+
     /** @test */
     public function testFetchMustacheMetasData()
     {