Skip to content

test: do not assume tls handshake order #25508

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sam-github wants to merge 11 commits into from
Closed

test: do not assume tls handshake order #25508

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
74f1db9
test: wait for TCP connect, not TLS handshake
sam-github Feb 1, 2019
6c13102
test: do not assume server gets secure connection
sam-github Feb 1, 2019
78565d7
test: do not assume tls handshake order
sam-github Jan 8, 2019
c523483
test: do not race connection and rejection
sam-github Jan 9, 2019
e981f20
test: pin regression test for #8074 to TLS 1.2
sam-github Jan 15, 2019
39327b2
test: end tls gracefully, rather than destroy
sam-github Feb 1, 2019
79412a4
test: send a bad record only after connection done
sam-github Feb 1, 2019
431fd20
test: use mustCall in ephemeralkeyinfo test
sam-github Feb 1, 2019
55e0809
test: use common.mustCall(), and log the events
sam-github Feb 1, 2019
3dab493
test: use mustCall(), not global state checks
sam-github Feb 1, 2019
619b034
test: clarify confusion over "client" in comment
sam-github Feb 1, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
test: clarify confusion over "client" in comment 
Fix perplexing comment. It's not that TLS "clients" don't support
'secureConnect', it's that client sockets created with `new TLSSocket`
(as opposed to `tls.connect()`) don't support that event.
  • Loading branch information
@sam-github
sam-github committed Feb 5, 2019
commit 619b034ecf4e134a9d859e7598fe0d465c067c52
5 changes: 3 additions & 2 deletions test/parallel/test-tls-socket-default-options.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,9 @@ function test(client, callback) {
}));
}));

// Client doesn't support the 'secureConnect' event, and doesn't error if
// authentication failed. Caller must explicitly check for failure.
// `new TLSSocket` doesn't support the 'secureConnect' event on client side,
// and doesn't error if authentication failed. Caller must explicitly check
// for failure.
(new tls.TLSSocket(null, client)).connect(pair.server.server.address().port)
.on('connect', common.mustCall(function() {
this.end('hello');
Expand Down