nodejs
diff --git a/Collapse file
‎lib/internal/http2/compat.js‎
Copy file name to clipboardExpand all lines: lib/internal/http2/compat.js
+25Lines changed: 25 additions & 0 deletions b/Collapse file
‎lib/internal/http2/compat.js‎
Copy file name to clipboardExpand all lines: lib/internal/http2/compat.js
+25Lines changed: 25 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-http2-server-set-header.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-http2-server-set-header.js
+7Lines changed: 7 additions & 0 deletions b/Collapse file
‎test/parallel/test-http2-server-set-header.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-http2-server-set-header.js
+7Lines changed: 7 additions & 0 deletions
@@ -45,6 +45,7 @@ const {
 } = constants;
 
 let statusMessageWarned = false;
+let statusConnectionHeaderWarned = false;
 
 // Defines and implements an API compatibility layer on top of the core
 // HTTP/2 implementation, intended to provide an interface that is as
@@ -58,6 +59,8 @@ function assertValidHeader(name, value) {
     err = new ERR_HTTP2_PSEUDOHEADER_NOT_ALLOWED();
   } else if (value === undefined || value === null) {
     err = new ERR_HTTP2_INVALID_HEADER_VALUE(value, name);
+  } else if (!isConnectionHeaderAllowed(name, value)) {
+    connectionHeaderMessageWarn();
   }
   if (err !== undefined) {
     Error.captureStackTrace(err, assertValidHeader);
@@ -88,6 +91,23 @@ function statusMessageWarn() {
   }
 }
 
+function isConnectionHeaderAllowed(name, value) {
+  return name !== constants.HTTP2_HEADER_CONNECTION ||
+         value === 'trailers';
+}
+
+function connectionHeaderMessageWarn() {
+  if (statusConnectionHeaderWarned === false) {
+    process.emitWarning(
+      'The provided connection header is not valid, ' +
+      'the value will be dropped from the header and ' +
+      'will never be in use.',
+      'UnsupportedWarning'
+    );
+    statusConnectionHeaderWarned = true;
+  }
+}
+
 function onStreamData(chunk) {
   const request = this[kRequest];
   if (request !== undefined && !request.push(chunk))
@@ -539,6 +559,11 @@ class Http2ServerResponse extends Stream {
   [kSetHeader](name, value) {
     name = name.trim().toLowerCase();
     assertValidHeader(name, value);
+
+    if (!isConnectionHeaderAllowed(name, value)) {
+      return;
+    }
+
     this[kHeaders][name] = value;
   }
 
 
@@ -11,6 +11,7 @@ const body =
 const server = http2.createServer((req, res) => {
   res.setHeader('foobar', 'baz');
   res.setHeader('X-POWERED-BY', 'node-test');
+  res.setHeader('connection', 'connection-test');
   res.end(body);
 });
 
@@ -34,4 +35,10 @@ server.listen(0, common.mustCall(() => {
   req.end();
 }));
 
+const compatMsg = 'The provided connection header is not valid, ' +
+                  'the value will be dropped from the header and ' +
+                  'will never be in use.';
+
+common.expectWarning('UnsupportedWarning', compatMsg);
+
 server.on('error', common.mustNotCall());