Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node-v0.x-archive
base: 85360f06db82
Choose a base ref
...
head repository: nodejs/node-v0.x-archive
compare: ae00e97ef27b
Choose a head ref
  • 1 commit
  • 1 file changed
  • 1 contributor

Commits on Jan 15, 2015

  1. test: fix test-crypto-stream.js 

    A recent merge of v0.10 to v0.12
(0c7f6ca) upgraded OpenSSL to version
1.0.1j. In v0.10, this required test-crypto-stream.js to be fixed with
commit 707cc25. Basically, instead of
returning the proper error, Err_get_error() would return 0 and the test
for the error message needed to be updated in test-crypto-stream.js.

However, in the v0.12 branch, crypto error messages are handled a bit
differently since commit 26a1b71
landed. Instead of returning the default OpenSSL error message, it makes
the decipher stream return a default message specific to Node.js.

This commit updates test-crypto-stream.js to test the error object
against the proper default error message.

Fixes #9019.

Reviewed-by: Trevor Norris <trev.norris@gmail.com>
    Julien Gilli committed Jan 15, 2015
    Copy the full SHA
    ae00e97 View commit details
    Browse the repository at this point in the history