lukechilds/node - node - Gitea: Git with a cup of tea

13096 Commits

131 Branches

446 Tags

325 MiB

Tree: bcec2fecbd

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
Shigeki Ohtsu	9dcc45e9c5	test: enable to work pkcs12 test in FIPS mode The pfx file created by pkcs12 command of openssl causes an error in FIPS mode because its certificate is encrypted with RC2 by default. Adding `-descert` option resolves the error. Fix: https://github.com/nodejs/node/pull/5144 Fix: https://github.com/nodejs/node/pull/5109 PR-URL: https://github.com/nodejs/node/pull/5150 Reviewed-By: Rich Trott <rtrott@gmail.com>	9 years ago
Fedor Indutny	e4390664ae	test: disable gh-5100 test when in FIPS mode This is a follow-up fix for half-broken test in `23196fe`, and an attempt to recover some dignity after breaking CI. PR-URL: https://github.com/nodejs/node/pull/5144 Reviewed-By: Rich Trott <rtrott@gmail.com>	9 years ago
Fedor Indutny	d9e934c71f	crypto: add `pfx` certs as CA certs too According to documentation all certificates specified in `pfx` option should be treated as a CA certificates too. While it doesn't seem to be logically correct to me, we can't afford to break API stability at this point. Fix: #5100 PR-URL: https://github.com/nodejs/node/pull/5109 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>	9 years ago

Shigeki Ohtsu

9dcc45e9c5

test: enable to work pkcs12 test in FIPS mode

The pfx file created by pkcs12 command of openssl causes an error in
FIPS mode because its certificate is encrypted with RC2 by default.
Adding `-descert` option resolves the error.

Fix: https://github.com/nodejs/node/pull/5144
Fix: https://github.com/nodejs/node/pull/5109
PR-URL: https://github.com/nodejs/node/pull/5150
Reviewed-By: Rich Trott <rtrott@gmail.com>

9 years ago

Fedor Indutny

e4390664ae

test: disable gh-5100 test when in FIPS mode

This is a follow-up fix for half-broken test in 23196fe, and an attempt
to recover some dignity after breaking CI.

PR-URL: https://github.com/nodejs/node/pull/5144
Reviewed-By: Rich Trott <rtrott@gmail.com>

9 years ago

Fedor Indutny

d9e934c71f

crypto: add `pfx` certs as CA certs too

According to documentation all certificates specified in `pfx` option
should be treated as a CA certificates too. While it doesn't seem to be
logically correct to me, we can't afford to break API stability at this
point.

Fix: #5100
PR-URL: https://github.com/nodejs/node/pull/5109
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>

9 years ago

3 Commits (bcec2fecbdb3e86fceff5c66678e4443666f78b9)