http2: fix premature destroy

Check stream._writableState.finished instead of stream.writable as the latter can lead to premature calls to destroy and dropped writes on busy processes. PR-URL: nodejs#21051 Fixes: nodejs#20750 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>
Trott · Jun 3, 2018 · 7b6c428 · 7b6c428
1 parent d528781
commit 7b6c428
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/internal/http2/core.js b/lib/internal/http2/core.js
@@ -1896,7 +1896,7 @@ class Http2Stream extends Duplex {
     }
 
     // TODO(mcollina): remove usage of _*State properties
-    if (!this.writable) {
+    if (this._writableState.finished) {
       if (!this.readable && this.closed) {
         this.destroy();
         return;