Skip to content

Commit

Permalink
test: remove common.PORT from test-tls-server-large-request
Browse files Browse the repository at this point in the history
Switch test-tls-server-large-request from common.PORT to a port assigned
by the operating system.

PR-URL: #31749
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Trott authored and codebytere committed Feb 27, 2020
1 parent e76ac1d commit 3fbd5ab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/pummel/test-tls-server-large-request.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,9 @@ const server = tls.Server(options, common.mustCall(function(socket) {
socket.pipe(mediator);
}));

server.listen(common.PORT, common.mustCall(function() {
server.listen(0, common.mustCall(() => {
const client1 = tls.connect({
port: common.PORT,
port: server.address().port,
rejectUnauthorized: false
}, common.mustCall(function() {
client1.end(request);
Expand Down

0 comments on commit 3fbd5ab

Please sign in to comment.