Do not throw when writing if socket closed

The call to shutdown() in runReadFiber will fail the request if the socket closes. The exception thrown in runWriteFiber was sometimes being caught by the event loop error handler as the top-level exception, not a previous exception as would be expected. I'm not sure how this is possible, but removing the throw fixed the issue.

Author: trowski

src/Connection/Internal/Http2ConnectionProcessor.php

@@ -100,6 +100,7 @@ final class Http2ConnectionProcessor implements Http2Processor
 
     private ?int $shutdown = null;
 
+    /** @var Queue<string> */
     private readonly Queue $frameQueue;
 
     public function __construct(
@@ -1597,11 +1598,16 @@ private function createStreamTimeoutWatcher(int $streamId, float $timeout, strin
     private function runWriteFiber(): void
     {
         try {
-            foreach ($this->frameQueue->iterate() as $frame) {
+            $iterator = $this->frameQueue->iterate();
+
+            while ($iterator->continue()) {
                 if (!$this->socket->isWritable()) {
-                    throw new SocketException('Connection has closed');
+                    $this->hasWriteError = true;
+                    $iterator->dispose();
+                    return;
                 }
 
+                $frame = $iterator->getValue();
                 $this->socket->write($frame);
             }
         } catch (\Throwable $exception) {
@@ -1610,7 +1616,7 @@ private function runWriteFiber(): void
             $this->shutdown(new SocketException(
                 "The HTTP/2 connection closed unexpectedly: " . $exception->getMessage(),
                 Http2Parser::INTERNAL_ERROR,
-                $exception
+                $exception,
             ));
         }
     }