crypto: add `pfx` certs as CA certs too
authorFedor Indutny <fedor@indutny.com>
Sat, 6 Feb 2016 04:13:30 +0000 (23:13 -0500)
committerMyles Borins <mborins@us.ibm.com>
Wed, 2 Mar 2016 22:01:11 +0000 (14:01 -0800)
commit689e1faaf11c9fe5b3e041116b3ab5e22dae664c
tree49ba8b911e24e748c2abcf079b190ea7edaa8d38
parent875a6d02ae480f928d6b8c618040c778b81a401b
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>
src/node_crypto.cc
test/parallel/test-tls-pfx-gh-5100-regr.js [new file with mode: 0644]