FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

http,https: give names to anonymous or misnamed functions · nodejs/node@cff440e · GitHub

/ node Public

Commit cff440e

Browse files
LiviaMedeiros
authored and
aduh95
committed
http,https: give names to anonymous or misnamed functions
Affected functions: - http.OutgoingMessage.prototype.cork - http.OutgoingMessage.prototype.uncork - http.Server.prototype.close - http.Server.prototype.closeAllConnections - http.Server.prototype.closeIdleConnections - http.Server.prototype[Symbol.asyncDispose] - http.Server.prototype[nodejs.rejection] - http.validateHeaderName - http.validateHeaderValue - https.Server.prototype.closeAllConnections - https.Server.prototype.closeIdleConnections - https.Server.prototype.close PR-URL: #58180 Reviewed-By: Paolo Insogna <paolo@cowtech.it> Reviewed-By: Ethan Arrowood <ethan@arrowood.dev> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 1db1c87 commit cff440e

3 files changed

Lines changed: 17 additions & 14 deletions

File tree

‎lib/_http_outgoing.js‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ const {
7171
hideStackFrames,
7272
} = require('internal/errors');
7373
const { validateString } = require('internal/validators');
74+
const { assignFunctionName } = internalUtil;
7475
const { isUint8Array } = require('internal/util/types');
7576

7677
let debug = require('internal/util/debuglog').debuglog('http', (fn) => {
@@ -312,14 +313,14 @@ OutgoingMessage.prototype._renderHeaders = function _renderHeaders() {
312313
return headers;
313314
};
314315

315-
OutgoingMessage.prototype.cork = function() {
316+
OutgoingMessage.prototype.cork = function cork() {
316317
this[kCorked]++;
317318
if (this[kSocket]) {
318319
this[kSocket].cork();
319320
}
320321
};
321322

322-
OutgoingMessage.prototype.uncork = function() {
323+
OutgoingMessage.prototype.uncork = function uncork() {
323324
this[kCorked]--;
324325
if (this[kSocket]) {
325326
this[kSocket].uncork();
@@ -664,21 +665,21 @@ function matchHeader(self, state, field, value) {
664665
}
665666
}
666667

667-
const validateHeaderName = hideStackFrames((name, label) => {
668+
const validateHeaderName = assignFunctionName('validateHeaderName', hideStackFrames((name, label) => {
668669
if (typeof name !== 'string' || !name || !checkIsHttpToken(name)) {
669670
throw new ERR_INVALID_HTTP_TOKEN.HideStackFramesError(label || 'Header name', name);
670671
}
671-
});
672+
}));
672673

673-
const validateHeaderValue = hideStackFrames((name, value) => {
674+
const validateHeaderValue = assignFunctionName('validateHeaderValue', hideStackFrames((name, value) => {
674675
if (value === undefined) {
675676
throw new ERR_HTTP_INVALID_HEADER_VALUE.HideStackFramesError(value, name);
676677
}
677678
if (checkInvalidHeaderChar(value)) {
678679
debug('Header "%s" contains invalid characters', name);
679680
throw new ERR_INVALID_CHAR.HideStackFramesError('header content', name);
680681
}
681-
});
682+
}));
682683

683684
function parseUniqueHeadersOption(headers) {
684685
if (!ArrayIsArray(headers)) {

‎lib/_http_server.js‎

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ const {
7979
},
8080
} = require('internal/errors');
8181
const {
82+
assignFunctionName,
8283
kEmptyObject,
8384
promisify,
8485
SymbolAsyncDispose,
@@ -573,17 +574,17 @@ function Server(options, requestListener) {
573574
ObjectSetPrototypeOf(Server.prototype, net.Server.prototype);
574575
ObjectSetPrototypeOf(Server, net.Server);
575576

576-
Server.prototype.close = function() {
577+
Server.prototype.close = function close() {
577578
httpServerPreClose(this);
578579
ReflectApply(net.Server.prototype.close, this, arguments);
579580
return this;
580581
};
581582

582-
Server.prototype[SymbolAsyncDispose] = async function() {
583+
Server.prototype[SymbolAsyncDispose] = assignFunctionName(SymbolAsyncDispose, async function() {
583584
return promisify(this.close).call(this);
584-
};
585+
});
585586

586-
Server.prototype.closeAllConnections = function() {
587+
Server.prototype.closeAllConnections = function closeAllConnections() {
587588
if (!this[kConnections]) {
588589
return;
589590
}
@@ -595,7 +596,7 @@ Server.prototype.closeAllConnections = function() {
595596
}
596597
};
597598

598-
Server.prototype.closeIdleConnections = function() {
599+
Server.prototype.closeIdleConnections = function closeIdleConnections() {
599600
if (!this[kConnections]) {
600601
return;
601602
}
@@ -618,7 +619,8 @@ Server.prototype.setTimeout = function setTimeout(msecs, callback) {
618619
return this;
619620
};
620621

621-
Server.prototype[EE.captureRejectionSymbol] = function(err, event, ...args) {
622+
Server.prototype[EE.captureRejectionSymbol] =
623+
assignFunctionName(EE.captureRejectionSymbol, function(err, event, ...args) {
622624
switch (event) {
623625
case 'request': {
624626
const { 1: res } = args;
@@ -639,7 +641,7 @@ Server.prototype[EE.captureRejectionSymbol] = function(err, event, ...args) {
639641
net.Server.prototype[SymbolFor('nodejs.rejection')]
640642
.apply(this, arguments);
641643
}
642-
};
644+
});
643645

644646
function checkConnections() {
645647
if (this.headersTimeout === 0 && this.requestTimeout === 0) {

‎lib/https.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ Server.prototype.closeIdleConnections = HttpServer.prototype.closeIdleConnection
110110

111111
Server.prototype.setTimeout = HttpServer.prototype.setTimeout;
112112

113-
Server.prototype.close = function() {
113+
Server.prototype.close = function close() {
114114
httpServerPreClose(this);
115115
ReflectApply(tls.Server.prototype.close, this, arguments);
116116
return this;

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL