ACP2E-1353: allow image/x-icon favicon; cr comments

Author: alzota

app/code/Magento/MediaStorage/Model/File/Validator/Image.php

@@ -8,12 +8,9 @@
 namespace Magento\MediaStorage\Model\File\Validator;
 
 use Laminas\Validator\AbstractValidator;
-use Magento\Framework\App\ObjectManager;
 use Magento\Framework\File\Mime;
 use Magento\Framework\Filesystem\Driver\File;
-use Magento\Framework\Image\Adapter\ConfigInterface;
 use Magento\Framework\Image\Factory;
-use Psr\Log\LoggerInterface;
 
 /**
  * Image validator
@@ -48,27 +45,19 @@ class Image extends AbstractValidator
      */
     private $file;
 
-    /**
-     * @var LoggerInterface
-     */
-    private $logger;
-
     /**
      * @param Mime $fileMime
      * @param Factory $imageFactory
      * @param File $file
-     * @param LoggerInterface|null $logger
      */
     public function __construct(
         Mime $fileMime,
         Factory $imageFactory,
-        File $file,
-        LoggerInterface $logger = null
+        File $file
     ) {
         $this->fileMime = $fileMime;
         $this->imageFactory = $imageFactory;
         $this->file = $file;
-        $this->logger = $logger ?? ObjectManager::getInstance()->get(LoggerInterface::class);
 
         parent::__construct();
     }
@@ -87,7 +76,6 @@ public function isValid($filePath): bool
                 $image->open();
             } catch (\Exception $e) {
                 $isValid = false;
-                $this->logger->critical($e, ['exception' => $e]);
             }
         }