You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
failed test: parallel/test-tls-no-sslv3.js with shared openssl #618
Path: parallel/test-tls-no-sslv3
child_process.js:1100
var err = this._handle.spawn(options);
^
TypeError: Bad argument
at TypeError (native)
at ChildProcess.spawn (child_process.js:1100:26)
at exports.spawn (child_process.js:959:9)
at Server.<anonymous> (parallel/test-tls-no-sslv3.js:25:16)
at Server.g (events.js:184:16)
at Server.emit (events.js:95:17)
at net.js:1167:12
at process._tickCallback (node.js:337:11)
at Function.Module.runMain (module.js:489:11)
at startup (node.js:111:16)
The problem is that common.opensslCli will return false instead of a path to the openssl-binary, since it hasn't been built. The simple way is just to check for false before running the test, but it feels like a poor decision since we'd probably want to run the test suite with a shared openssl as much as possible (so the user can verify that their setup is correct).
@jbergstroem Thanks for your report. This error comes from the change of an error message in openssl-1.0.2 and it is to be fixed with shigeki@72031e6 after upgrading to openssl-1.0.2 . Please wait for a moment until I finish it.
Activity
shigeki commentedon Feb 9, 2015
Fixed in #762
jbergstroem commentedon Feb 12, 2015
Here's the result from the test after fixing path to openssl binary:
shigeki commentedon Feb 12, 2015
@jbergstroem Thanks for your report. This error comes from the change of an error message in openssl-1.0.2 and it is to be fixed with shigeki@72031e6 after upgrading to openssl-1.0.2 . Please wait for a moment until I finish it.
jbergstroem commentedon Feb 12, 2015
@shigeki confirmed -- just tried with openssl 1.0.1l. I'm closing since we shouldn't assume 1.0.2 is working until that work is merged.