ACPT-1034: Fixing Logging fails when log rotated in long running processes

Author: JacobBrownAustin

lib/internal/Magento/Framework/Logger/Handler/Base.php

@@ -9,6 +9,7 @@
 
 use InvalidArgumentException;
 use Magento\Framework\Filesystem\DriverInterface;
+use Magento\Framework\ObjectManager\ResetAfterRequestInterface;
 use Monolog\Formatter\LineFormatter;
 use Monolog\Handler\StreamHandler;
 use Monolog\Logger;
@@ -18,7 +19,7 @@
  *
  * @api
  */
-class Base extends StreamHandler
+class Base extends StreamHandler implements ResetAfterRequestInterface
 {
     /**
      * @var string
@@ -59,6 +60,14 @@ public function __construct(
         $this->setFormatter(new LineFormatter(null, null, true));
     }
 
+    /**
+     * @inheritDoc
+     */
+    public function _resetState(): void
+    {
+        $this->close();
+    }
+
     /**
      * Remove dots from file name
      *

lib/internal/Magento/Framework/Logger/LoggerProxy.php

@@ -10,21 +10,22 @@
 use Magento\Framework\Exception\FileSystemException;
 use Magento\Framework\Exception\RuntimeException;
 use Magento\Framework\ObjectManager\NoninterceptableInterface;
+use Magento\Framework\ObjectManager\ResetAfterRequestInterface;
 use Magento\Framework\ObjectManagerInterface;
 use Psr\Log\LoggerInterface;
 
 /**
  * Create and use Logger implementation based on deployment configuration
  */
-class LoggerProxy implements LoggerInterface, NoninterceptableInterface
+class LoggerProxy implements LoggerInterface, NoninterceptableInterface, ResetAfterRequestInterface
 {
     /**
      * @var ObjectManagerInterface
      */
-    private $objectManager;
+    private readonly ObjectManagerInterface $objectManager;
 
     /**
-     * @var LoggerInterface
+     * @var LoggerInterface|null
      */
     private $logger;
 
@@ -39,6 +40,11 @@ public function __construct(
         $this->objectManager = $objectManager;
     }
 
+    public function _resetState(): void
+    {
+        $this->logger = null;
+    }
+
     /**
      * Remove links to other objects.
      *