nodejs
diff --git a/Collapse file
‎doc/api/http.md‎
Copy file name to clipboardExpand all lines: doc/api/http.md
+6-1Lines changed: 6 additions & 1 deletion
Display the source diff
Display the rich diff b/Collapse file
‎doc/api/http.md‎
Copy file name to clipboardExpand all lines: doc/api/http.md
+6-1Lines changed: 6 additions & 1 deletion
Display the source diff
Display the rich diff
diff --git a/Collapse file
‎lib/_http_outgoing.js‎
Copy file name to clipboardExpand all lines: lib/_http_outgoing.js
+3-5Lines changed: 3 additions & 5 deletions b/Collapse file
‎lib/_http_outgoing.js‎
Copy file name to clipboardExpand all lines: lib/_http_outgoing.js
+3-5Lines changed: 3 additions & 5 deletions
@@ -3657,13 +3657,18 @@ Passing an `AbortSignal` and then calling `abort` on the corresponding
 `AbortController` will behave the same way as calling `.destroy()` on the
 request itself.
 
-## `http.validateHeaderName(name)`
+## `http.validateHeaderName(name[, label])`
 
 <!-- YAML
 added: v14.3.0
+changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/46143
+    description: The `label` parameter is added.
 -->
 
 * `name` {string}
+* `label` {string} Label for error message. **Default:** `'Header name'`.
 
 Performs the low-level validations on the provided `name` that are done when
 `res.setHeader(name, value)` is called.
 
@@ -627,9 +627,9 @@ function matchHeader(self, state, field, value) {
   }
 }
 
-const validateHeaderName = hideStackFrames((name) => {
+const validateHeaderName = hideStackFrames((name, label) => {
   if (typeof name !== 'string' || !name || !checkIsHttpToken(name)) {
-    throw new ERR_INVALID_HTTP_TOKEN('Header name', name);
+    throw new ERR_INVALID_HTTP_TOKEN(label || 'Header name', name);
   }
 });
 
@@ -932,9 +932,7 @@ OutgoingMessage.prototype.addTrailers = function addTrailers(headers) {
       field = key;
       value = headers[key];
     }
-    if (typeof field !== 'string' || !field || !checkIsHttpToken(field)) {
-      throw new ERR_INVALID_HTTP_TOKEN('Trailer name', field);
-    }
+    validateHeaderName(field, 'Trailer name');
 
     // Check if the field must be sent several times
     const isArrayValue = ArrayIsArray(value);
Original file line number	Diff line number	Diff line change
`@@ -627,9 +627,9 @@ function matchHeader(self, state, field, value) {`
`627`	`627`	`}`
`628`	`628`	`}`
`629`	`629`
`630`		`-const validateHeaderName = hideStackFrames((name) => {`
	`630`	`+const validateHeaderName = hideStackFrames((name, label) => {`
`631`	`631`	`if (typeof name !== 'string' \|\| !name \|\| !checkIsHttpToken(name)) {`
`632`		`- throw new ERR_INVALID_HTTP_TOKEN('Header name', name);`
	`632`	`+ throw new ERR_INVALID_HTTP_TOKEN(label \|\| 'Header name', name);`
`633`	`633`	`}`
`634`	`634`	`});`
`635`	`635`
`@@ -932,9 +932,7 @@ OutgoingMessage.prototype.addTrailers = function addTrailers(headers) {`
`932`	`932`	`field = key;`
`933`	`933`	`value = headers[key];`
`934`	`934`	`}`
`935`		`- if (typeof field !== 'string' \|\| !field \|\| !checkIsHttpToken(field)) {`
`936`		`- throw new ERR_INVALID_HTTP_TOKEN('Trailer name', field);`
`937`		`- }`
	`935`	`+ validateHeaderName(field, 'Trailer name');`
`938`	`936`
`939`	`937`	`// Check if the field must be sent several times`
`940`	`938`	`const isArrayValue = ArrayIsArray(value);`