Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: set clientOpts.port property #19767

Closed
wants to merge 2 commits into from
Closed

test: set clientOpts.port property #19767

wants to merge 2 commits into from

Commits on Apr 13, 2018

  1. test: set clientOpts.port property 

    Currently this test will overwrite the clientOpts object with the port,
instead of setting the port property on the clientOpts object which
looks like the original intent.

Doing this the test fails reporting that the fake-cnnic-root-cert has
expired. This is indeed true:
$ openssl x509 -in test/fixtures/keys/fake-cnnic-root-cert.pem \
-text -noout
Certificate:
        ...
        Validity
            Not Before: Jun  9 17:15:16 2015 GMT
            Not After : Mar 29 17:15:16 2018 GMT

This commit sets the errorCode to CERT_HAS_EXPIRED. I tried updating the
certificate using test/fixtures/keys/Makefile but then no error is
thrown and I'm currently looking into this.
    @danbev
    danbev committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    07ca62c View commit details
    Browse the repository at this point in the history

  2. test: remove test case 0 from tls-cnnic-whitelist 

    I looks like this test has not worked as expected since commit
2bc7841 ("test: use random ports
where possible"). The test in that commit checked for `CERT_REVOKED`
which was returned by CheckWhitelistedServerCert.

CheckWhitelistedServerCert was later removed in commit
dc87543 ("src: drop CNNIC+StartCom
certificate whitelisting").

I'm suggesting that this test case be removed as I don't think it is
valid anymore.
    @danbev
    danbev committed Apr 13, 2018
    Configuration menu
    Copy the full SHA
    5753046 View commit details
    Browse the repository at this point in the history