delta/node-new.git/test/parallel/test-https-foafssl.js, branch devsnek-patch-1

test: add debugging for callbacks in test-https-foafssl.js

2020-08-07T15:41:07+00:00

The test is timing out once in a very long while on Windows CI. It is
unclear where the test gets stuck, so add more debugging statements to
try to locate it.

Refs: https://github.com/nodejs/node/issues/24397

PR-URL: https://github.com/nodejs/node/pull/34603
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell

test: add debugging for test-https-foafssl.js

2020-08-07T15:41:05+00:00

The test is timing out once in a very long while on Windows CI. It is
unclear where the test gets stuck, so add more debugging statements to
try to locate it.

Refs: https://github.com/nodejs/node/issues/24397

PR-URL: https://github.com/nodejs/node/pull/34603
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell

test: move foafssl certs to fixtures/keys/

2019-06-10T16:57:23+00:00

PR-URL: https://github.com/nodejs/node/pull/27962
Reviewed-By: Sam Roberts 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Rich Trott

Revert "test: move all test keys/certs under `test/fixtures/keys/`"

2019-06-05T23:01:13+00:00

PR-URL: https://github.com/nodejs/node/pull/28083
Reviewed-By: Anna Henningsen 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Refael Ackermann (רפאל פלחי)

test: move foafssl certs to fixtures/keys/

2019-06-05T18:37:52+00:00

PR-URL: https://github.com/nodejs/node/pull/27962
Reviewed-By: Sam Roberts 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Rich Trott

test: remove openssl -no_rand_screen opts

2018-04-09T21:45:45+00:00

This option was made from the floating patch of d269e07.
It is no longer needed because the issue was resolved in
OpenSSL-1.1.0.

Fixes: https://github.com/nodejs/node/issues/4270
PR-URL: https://github.com/nodejs/node/pull/19794
Reviewed-By: James M Snell 
Reviewed-By: Rod Vagg 
Reviewed-By: Michael Dawson

test: use common fixtures module

2017-10-12T20:36:55+00:00

Replace use of fixturesDir with fixtures.readSync and fixtures.path.

PR-URL: https://github.com/nodejs/node/pull/15856
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca

test: simplify test skipping

2017-07-04T09:41:49+00:00

* 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

test: reduce string concatenations

2017-05-05T14:39:05+00:00

PR-URL: https://github.com/nodejs/node/pull/12735
Refs: https://github.com/nodejs/node/pull/12455
Reviewed-By: Refael Ackermann 
Reviewed-By: Gibson Fahnestock

tls: make rejectUnauthorized default to true

2017-03-23T20:27:56+00:00

rejectUnauthorized used to be false when the property was undefined or
null, quietly allowing client connections for which certificates have
been requested (requestCert is true) even when the client certificate
was not authorized (signed by a trusted CA). Change this so
rejectUnauthorized is always true unless it is explicitly set to false.

PR-URL: https://github.com/nodejs/node/pull/5923
Reviewed-By: Sam Roberts 
Reviewed-By: James M Snell 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig