http2: skip writeHead if stream is closed

Fixes: #57416
PR-URL: #57686
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>

Author: islandryu

lib/internal/http2/compat.js

@@ -706,7 +706,7 @@ class Http2ServerResponse extends Stream {
 writeHead(statusCode,statusMessage,headers){
 conststate=this[kState];
 
-if(state.closed||this.stream.destroyed)
+if(state.closed||this.stream.destroyed||this.stream.closed)
 returnthis;
 if(this[kStream].headersSent)
 thrownewERR_HTTP2_HEADERS_SENT();

test/parallel/test-http2-compat-write-head-after-close.js

@@ -0,0 +1,23 @@
+'use strict';
+
+constcommon=require('../common');
+if(!common.hasCrypto){common.skip('missing crypto');};
+consth2=require('http2');
+
+constserver=h2.createServer((req,res)=>{
+conststream=req.stream;
+stream.close();
+res.writeHead(200,{'content-type': 'text/plain'});
+});
+
+server.listen(0,common.mustCall(()=>{
+constport=server.address().port;
+constclient=h2.connect(`http://localhost:${port}`);
+constreq=client.request({':path': '/'});
+req.on('response',common.mustNotCall('head after close should not be sent'));
+req.on('end',common.mustCall(()=>{
+client.close();
+server.close();
+}));
+req.end();
+}));