fix: only set keep-alive header before Node.js 14.8.0

Author: atian25

app/middleware/meta.js

@@ -4,6 +4,8 @@
 
 'use strict';
 
+const semver = require('semver');
+
 module.exports = options => {
   return async function meta(ctx, next) {
     if (options.logging) {
@@ -13,9 +15,12 @@ module.exports = options => {
     // total response time header
     ctx.set('x-readtime', Date.now() - ctx.starttime);
 
-    // try to support Keep-Alive Header
+    // Node.js >=14.8.0 will set Keep-Alive Header, see https://github.com/nodejs/node/pull/34561
+    const shouldPatchKeepAliveHeader = semver.lt(process.version, '14.8.0');
+
+    // try to support Keep-Alive Header when < 14.8.0
     const server = ctx.app.server;
-    if (server && server.keepAliveTimeout && server.keepAliveTimeout >= 1000 && ctx.header.connection !== 'close') {
+    if (shouldPatchKeepAliveHeader && server && server.keepAliveTimeout && server.keepAliveTimeout >= 1000 && ctx.header.connection !== 'close') {
       /**
        * use Math.floor instead of parseInt. More: https://github.com/eggjs/egg/pull/2702
        */

package.json

@@ -49,6 +49,7 @@
     "ms": "^2.1.1",
     "mz": "^2.7.0",
     "on-finished": "^2.3.0",
+    "semver": "^7.3.2",
     "sendmessage": "^1.1.0",
     "urllib": "^2.33.0",
     "utility": "^1.15.0",