Merge pull request #3153 from cloudflare/jsnell/no-need-to-error-on-a…

…lready-closing-writable
cloudflare · Nov 22, 2024 · c4f6890 · c4f6890
2 parents 0aa984a + 64478a0
commit c4f6890
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/workerd/api/streams/internal.c++ b/src/workerd/api/streams/internal.c++
@@ -1020,8 +1020,7 @@ void WritableStreamInternalController::setHighWaterMark(uint64_t highWaterMark)
 
 jsg::Promise<void> WritableStreamInternalController::closeImpl(jsg::Lock& js, bool markAsHandled) {
   if (isClosedOrClosing()) {
-    auto reason = js.v8TypeError("This WritableStream has been closed."_kj);
-    return rejectedMaybeHandledPromise<void>(js, reason, markAsHandled);
+    return js.resolvedPromise();
   }
   if (isPiping()) {
     auto reason = js.v8TypeError("This WritableStream is currently being piped to."_kj);