summaryrefslogtreecommitdiff
path: root/test/parallel/test-tls-wrap-econnreset-socket.js
Commit message (Collapse)AuthorAgeFilesLines
* stream: fix multiple destroy callsRobert Nagy2020-02-291-0/+5
| | | | | | | | | | | | | | | Previously destroy could be called multiple times causing inconsistent and hard to predict behavior. Furthermore, since the stream _destroy implementation can only be called once, the behavior of applying destroy multiple times becomes unclear. This changes so that only the first destroy() call is executed and any subsequent calls are noops. PR-URL: https://github.com/nodejs/node/pull/29197 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
* test: simplify test skippingVse Mozhet Byt2017-07-041-3/+2
| | | | | | | | | | | | * Make common.skip() exit. Also add common.printSkipMessage() for partial skips. * Don't make needless things before skip PR-URL: https://github.com/nodejs/node/pull/14021 Fixes: https://github.com/nodejs/node/issues/14016 Reviewed-By: Refael Ackermann <refack@gmail.com>
* test: add crypto check to test-tls-wrap-econnresetDaniel Bevenius2017-06-171-0/+4
| | | | | | | | | | | | | | | Currently, there are a few test-tls-wrap-econnreset test that fail when Node is configured --without-ssl: Error: Node.js is not compiled with openssl crypto support This commit adds crypto checks and skips these tests if no crypto support unavailable. PR-URL: https://github.com/nodejs/node/pull/13691 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* tls: add host and port info to ECONNRESET errorsJoseĢ F. Romaniello2017-06-141-0/+26
Add more information to the "ECONNRESET" errors generated when the socket hang ups before establishing the secure connection. These kind of errors are really hard to troubleshoot without this info. PR-URL: https://github.com/nodejs/node/pull/7476 Reviewed-By: Trevor Norris <trevnorris@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Yazhong Liu <yorkiefixer@gmail.com>
View Lorry Controller Status