GraphQl-220: Implement exception logging.

Removed client errors logging.

Author: Maksym Novik

app/etc/graphql/di.xml

@@ -7,29 +7,21 @@
 -->
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
     <preference for="Magento\Framework\GraphQl\Query\ErrorHandlerInterface" type="Magento\Framework\GraphQl\Query\ErrorHandler"/>
-    <preference for="Magento\Framework\GraphQl\Query\Resolver\LoggerFactoryInterface" type="Magento\Framework\GraphQl\Query\Resolver\LoggerFactory"/>
-    <virtualType name="GraphQLClientLogger" type="Magento\Framework\Logger\Monolog">
+    <virtualType name="GraphQlLogger" type="Magento\Framework\Logger\Monolog">
         <arguments>
             <argument name="handlers" xsi:type="array">
-                <item name="error" xsi:type="object">GraphQLClientErrorHandler</item>
+                <item name="error" xsi:type="object">GraphQlErrorHandler</item>
             </argument>
         </arguments>
     </virtualType>
-    <virtualType name="GraphQLClientErrorHandler" type="Magento\Framework\Logger\Handler\Base">
+    <virtualType name="GraphQlErrorHandler" type="Magento\Framework\Logger\Handler\Base">
         <arguments>
-            <argument name="fileName" xsi:type="const">Magento\Framework\GraphQl\Query\ErrorHandler::CLIENT_LOG_FILE</argument>
+            <argument name="fileName" xsi:type="string">var/log/graphql/exception.log</argument>
         </arguments>
     </virtualType>
-    <virtualType name="GraphQLServerLogger" type="Magento\Framework\Logger\Monolog">
+    <type name="Magento\Framework\GraphQl\Query\ErrorHandler">
         <arguments>
-            <argument name="handlers" xsi:type="array">
-                <item name="error" xsi:type="object">GraphQLServerErrorHandler</item>
-            </argument>
-        </arguments>
-    </virtualType>
-    <virtualType name="GraphQLServerErrorHandler" type="Magento\Framework\Logger\Handler\Base">
-        <arguments>
-            <argument name="fileName" xsi:type="const">Magento\Framework\GraphQl\Query\ErrorHandler::SERVER_LOG_FILE</argument>
+            <argument name="logger" xsi:type="object">GraphQlLogger</argument>
         </arguments>
-    </virtualType>
+    </type>
 </config>

lib/internal/Magento/Framework/GraphQl/Query/ErrorHandler.php

@@ -8,7 +8,7 @@
 namespace Magento\Framework\GraphQl\Query;
 
 use GraphQL\Error\ClientAware;
-use Magento\Framework\GraphQl\Query\Resolver\LoggerFactoryInterface;
+use Psr\Log\LoggerInterface;
 
 /**
  * @inheritDoc
@@ -17,21 +17,18 @@
  */
 class ErrorHandler implements ErrorHandlerInterface
 {
-    const SERVER_LOG_FILE = 'var/log/graphql/server/exception.log';
-    const CLIENT_LOG_FILE = 'var/log/graphql/client/exception.log';
-
     /**
-     * @var LoggerFactoryInterface
+     * @var LoggerInterface
      */
-    private $loggerFactory;
+    private $logger;
 
     /**
-     * @param LoggerFactoryInterface $loggerFactory
+     * @param LoggerInterface $logger
      */
     public function __construct(
-        LoggerFactoryInterface $loggerFactory
+        LoggerInterface $logger
     ) {
-        $this->loggerFactory = $loggerFactory;
+        $this->logger = $logger;
     }
 
     /**
@@ -41,7 +38,7 @@ public function handle(array $errors, callable $formatter): array
     {
         return array_map(
             function (ClientAware $error) use ($formatter) {
-                $this->loggerFactory->getLogger($error)->error($error);
+                $this->logger->error($error);
 
                 return $formatter($error);
             },