Throw FOSCKEditorBundle exception on bad proxy url (#176)

src/Exception/BadProxyUrlException.php

@@ -0,0 +1,26 @@
+<?php
+
+/*
+ * This file is part of the FOSCKEditor Bundle.
+ *
+ * (c) 2018 - present  Friends of Symfony
+ * (c) 2009 - 2017     Eric GELOEN <[email protected]>
+ *
+ * For the full copyright and license information, please read the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace FOS\CKEditorBundle\Exception;
+
+use RuntimeException;
+
+/**
+ * @author Marko Kunic <[email protected]>
+ */
+final class BadProxyUrlException extends RuntimeException implements FOSCKEditorException
+{
+    public static function fromEnvUrl(string $url): self
+    {
+        return new static(sprintf('Unable to parse provided proxy url "%s".', $url));
+    }
+}

src/Installer/CKEditorInstaller.php

@@ -12,6 +12,7 @@
 
 namespace FOS\CKEditorBundle\Installer;
 
+use FOS\CKEditorBundle\Exception\BadProxyUrlException;
 use Symfony\Component\OptionsResolver\Options;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
@@ -186,6 +187,11 @@ private function createStreamContext(callable $notifier = null)
 
         if ($proxy) {
             $proxyUrl = parse_url($proxy);
+
+            if (false === $proxyUrl || !isset($proxyUrl['host']) || !isset($proxyUrl['port'])) {
+                throw BadProxyUrlException::fromEnvUrl($proxy);
+            }
+
             $context['http'] = [
                 'proxy' => 'tcp://'.$proxyUrl['host'].':'.$proxyUrl['port'],
                 'request_fulluri' => (bool) getenv('https_proxy_request_fulluri') ?:

tests/Installer/CKEditorInstallerTest.php

@@ -12,6 +12,7 @@
 
 namespace FOS\CKEditorBundle\Tests\Installer;
 
+use FOS\CKEditorBundle\Exception\BadProxyUrlException;
 use FOS\CKEditorBundle\Installer\CKEditorInstaller;
 use PHPUnit\Framework\TestCase;
 
@@ -143,6 +144,28 @@ public function testInstallWithHttpsProxyRequestFullUri(): void
         $this->assertInstall();
     }
 
+    /**
+     * @group proxy
+     */
+    public function testInstallWithProxyUrlMissingHost(): void
+    {
+        putenv('http_proxy=notgonnahappen');
+
+        $this->expectException(BadProxyUrlException::class);
+        $this->installer->install();
+    }
+
+    /**
+     * @group proxy
+     */
+    public function testInstallWithProxyUrlMissingPort(): void
+    {
+        putenv('http_proxy=http://notgonnahappen.com');
+
+        $this->expectException(BadProxyUrlException::class);
+        $this->installer->install();
+    }
+
     public function testReinstall(): void
     {
         $this->installer->install();