http2: emit timeout on compat request and response

v8.x Backport Note: The timeout has been increased to 10ms.
Fixes: #20079
Backport-PR-URL: #22850
PR-URL: #22252
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: jasnell

lib/internal/http2/compat.js

@@ -229,6 +229,13 @@ function onStreamCloseRequest() {
 req.emit('close');
 }
 
+functiononStreamTimeout(kind){
+returnfunctiononStreamTimeout(){
+constobj=this[kind];
+obj.emit('timeout');
+};
+}
+
 classHttp2ServerRequestextendsReadable{
 constructor(stream,headers,options,rawHeaders){
 super(options);
@@ -251,6 +258,7 @@ class Http2ServerRequest extends Readable {
 stream.on('error',onStreamError);
 stream.on('aborted',onStreamAbortedRequest);
 stream.on('close',onStreamCloseRequest);
+stream.on('timeout',onStreamTimeout(kRequest));
 this.on('pause',onRequestPause);
 this.on('resume',onRequestResume);
 }
@@ -403,6 +411,7 @@ class Http2ServerResponse extends Stream {
 stream.on('aborted',onStreamAbortedResponse);
 stream.on('close',onStreamCloseResponse);
 stream.on('wantTrailers',onStreamTrailersReady);
+stream.on('timeout',onStreamTimeout(kResponse));
 }
 
 // User land modules such as finalhandler just check truthiness of this

test/parallel/test-http2-compat-serverrequest-settimeout.js

@@ -6,13 +6,15 @@ if (!common.hasCrypto)
 constassert=require('assert');
 consthttp2=require('http2');
 
-constmsecs=common.platformTimeout(1);
+// Set the timeout to 10ms since ending the response stream resets the timer.
+constmsecs=common.platformTimeout(10);
 constserver=http2.createServer();
 
 server.on('request',(req,res)=>{
 req.setTimeout(msecs,common.mustCall(()=>{
 res.end();
 }));
+res.on('timeout',common.mustCall());
 res.on('finish',common.mustCall(()=>{
 req.setTimeout(msecs,common.mustNotCall());
 process.nextTick(()=>{

test/parallel/test-http2-compat-serverresponse-settimeout.js

@@ -6,13 +6,15 @@ if (!common.hasCrypto)
 constassert=require('assert');
 consthttp2=require('http2');
 
-constmsecs=common.platformTimeout(1);
+// Set the timeout to 10ms since ending the response stream resets the timer.
+constmsecs=common.platformTimeout(10);
 constserver=http2.createServer();
 
 server.on('request',(req,res)=>{
 res.setTimeout(msecs,common.mustCall(()=>{
 res.end();
 }));
+res.on('timeout',common.mustCall());
 res.on('finish',common.mustCall(()=>{
 res.setTimeout(msecs,common.mustNotCall());
 process.nextTick(()=>{