Fix for termination request handling

Author: Paramtamtam

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog][keepachangelog] and this project adheres to [Semantic Versioning][semver].
 
+## v4.0.1
+
+### Fixed
+
+- Termination request handling (safe loop breaking)
+
 ## v4.0.0
 
 ### Added

src/Worker.php

@@ -4,6 +4,7 @@
 
 namespace Spiral\RoadRunnerLaravel;
 
+use Psr\Http\Message\ServerRequestInterface;
 use Throwable;
 use Illuminate\Http\Request;
 use InvalidArgumentException;
@@ -51,6 +52,10 @@ public function start(WorkerOptionsInterface $options): void
         $this->fireEvent($app, new Events\BeforeLoopStartedEvent($app));
 
         while ($req = $psr7_worker->waitRequest()) {
+            if (!($req instanceof ServerRequestInterface)) { // termination request received
+                break;
+            }
+
             $responded = false;
 
             if ($options->getRefreshApp()) {

tests/Feature/WorkerTest.php

@@ -100,9 +100,6 @@ protected function createConfigFile(string $where, string $relay_type): string
   static:
     dir: "{$static_dir}"
     forbid: [".php"]
-
-endure:
-  log_level: warning # do NOT comment this lines - test will fails otherwise O_o
 EOF;
 
         $wrote = (new Filesystem())->put($config_file_path, $content, true);
@@ -148,7 +145,9 @@ public function testServerStartingUsingDifferentRelayTypes(): void
             )]);
             $http_client = new \GuzzleHttp\Client(['base_uri' => 'http://127.0.0.1:22622']);
 
-            // $rr_proc->disableOutput(); // @todo: This will fix `endure.log_level` problem, thx 2 @jetexe
+            // is required for preventing child process blocking, thx @jetexe
+            // more info: <https://www.php.net/manual/ru/function.proc-open.php#38870>
+            $rr_proc->disableOutput();
 
             try {
                 // https://symfony.com/doc/current/components/process.html#running-processes-asynchronously