Merge pull request #60 from tamer-badawy/master

use curl insted of file_get_content if available
thephpleague · Sep 24, 2022 · 552226f · 552226f
2 parents 3c6e46c + 64b3544
commit 552226f
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/composer.json b/composer.json
@@ -24,6 +24,9 @@
         "friendsofphp/php-cs-fixer": "~2",
         "phpunit/phpunit": "^9.5"
     },
+    "suggest": {
+        "ext-curl": "To download images from remote URLs if allow_url_fopen is disabled for security reasons"
+    },
     "autoload": {
         "psr-4": {
             "League\\ColorExtractor\\": "src"

diff --git a/src/Palette.php b/src/Palette.php
@@ -63,6 +63,35 @@ public static function fromFilename($filename, $backgroundColor = null)
         return self::fromContents(file_get_contents($filename), $backgroundColor);
     }
 
+    /**
+     * @param string   $url
+     * @param int|null $backgroundColor
+     *
+     * @return Palette
+     *
+     * @throws \RuntimeException
+     */
+    public static function fromUrl($url, $backgroundColor = null)
+    {
+        if (!function_exists('curl_init')){
+            return self::fromContents(file_get_contents($url));
+        }
+
+        $ch = curl_init();
+        try {
+            curl_setopt($ch, CURLOPT_URL, $url);
+            curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
+            $contents = curl_exec($ch);
+            if ($contents === false) {
+                throw new \RuntimeException('Failed to fetch image from URL');
+            }
+        } finally {
+            curl_close($ch);
+        }
+
+        return self::fromContents($contents, $backgroundColor);
+    }
+
     /**
      * Create instance with file contents
      *