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

simple/test-tls-server-verify failing on v0.10 #7963

Closed
isaacs opened this issue Jul 17, 2014 · 3 comments
Closed

simple/test-tls-server-verify failing on v0.10 #7963

isaacs opened this issue Jul 17, 2014 · 3 comments

Comments

@isaacs
Copy link

isaacs commented Jul 17, 2014

$ python tools/test.py simple/test-tls-server-verify
=== release test-tls-server-verify ===
Path: simple/test-tls-server-verify
connecting with agent1
  connecting with agent2
  connecting with agent3
  connecting with nocert
  connecting with agent1
  connecting with agent2
  connecting with agent3
  connecting with nocert
  connecting with agent1
  connecting with agent2
  connecting with agent3
  connecting with nocert
  connecting with agent1
  connecting with agent2
  connecting with agent3
  connecting with nocert
  connecting with agent1
  connecting with agent2
  connecting with agent3
Running 'Do not request certs. Everyone is unauthorized.'
- unauthed connection: undefined
  * unauthed
- unauthed connection: undefined
  * unauthed
- unauthed connection: undefined
  * unauthed
- unauthed connection: undefined
  * unauthed
Running 'Allow both authed and unauthed connections with CA1'
- authed connection: agent1
  * authed
- unauthed connection: DEPTH_ZERO_SELF_SIGNED_CERT
  * unauthed
- unauthed connection: UNABLE_TO_VERIFY_LEAF_SIGNATURE
  * unauthed
- unauthed connection: UNABLE_TO_GET_ISSUER_CERT
  * unauthed
Running 'Allow only authed connections with CA1'
- authed connection: agent1
  * authed
Running 'Allow only authed connections with CA1 and CA2'
- authed connection: agent1
  * authed
- authed connection: agent3
  * authed
Running 'Allow only certs signed by CA2 but not in the CRL'

assert.js:92
  throw new assert.AssertionError({
        ^
AssertionError: agent3 rejected, but should NOT have been
    at ChildProcess.<anonymous> (/Users/isaacs/dev/js/node-v0.10/test/simple/test-tls-server-verify.js:217:14)
    at ChildProcess.emit (events.js:98:17)
    at Process.ChildProcess._handle.onexit (child_process.js:809:12)
Command: out/Release/node /Users/isaacs/dev/js/node-v0.10/test/simple/test-tls-server-verify.js
[00:00|% 100|+   0|-   1]: Done

Bisecting now.

@indutny
Copy link
Member

indutny commented Jul 17, 2014

[edit] Running it on mac?

@isaacs
Copy link
Author

isaacs commented Jul 17, 2014

Yes, this is on a mac. Seems to be broken in 0.10 back quite a ways.

indutny added a commit that referenced this issue Jul 23, 2014
@indutny
Copy link
Member

indutny commented Jul 23, 2014

Fixed in 93390ff, thank you.

@indutny indutny closed this as completed Jul 23, 2014
mscdex pushed a commit to mscdex/node that referenced this issue Dec 25, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants