Skip to content
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

Jump to bottom

test: fix tls-inception #4195

Closed
wants to merge 2 commits into from
Closed

test: fix tls-inception #4195

wants to merge 2 commits into from

Commits on Dec 8, 2015

  1. test: fix tls-inception 

    Make sure all the data is read before checking its validity.
Remove `gotHello` variable and just check that the ssl `end` event
is received.
Remove unused variables.
    @santigimeno
    santigimeno committed Dec 8, 2015
    Configuration menu
    Copy the full SHA
    89ff7dd View commit details
    Browse the repository at this point in the history

  2. test: fix tls-inception flakiness 

    When sending a very large buffer (400000 bytes) the test fails due to
the client socket from the `a` server erroring with `ECONNRESET`.
There's a race condition between the closing of this socket and the `ssl`
socket closing on the other side of the connection. To improve things,
destroy the socket as soon as possible: in the `end` event of the `dest`
socket.
    @santigimeno
    santigimeno committed Dec 8, 2015
    Configuration menu
    Copy the full SHA
    432a86e View commit details
    Browse the repository at this point in the history