-
Notifications
You must be signed in to change notification settings - Fork 393
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.
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
Errors with docker test script #329
Comments
Generally, the Docker tests will spin up containers for your newly-
built sslscan to run against. The output will be diff'ed against the
expected output, and any variation will cause a failure. When new
functionality is added, the expected results may change, and hence, the
tests must be updated. Of course, failures can also be the result of
new bugs introduced by the code update. In this case, since you're
only removing supposedly dead code from the code base, the expected
results should all be the same.
Test #13 failed (among other reasons) because TLS v1.0 and v1.1 are
enabled on the test endpoint, but your sslscan build thinks they're
disabled. Also, strangely, your sslscan build is detecting that x448
is enabled in TLS v1.2 and v1.3, even though the master branch does
not.
|
My point was this was against the HEAD of the master branch without any local changes at all. My expectation would be there are no failed tests, but when I run it, the output I'm seeing is what I've put in this issue. Any pointers on where to look at this? Am I the only one seeing this? For reference, I'm using an Ubuntu 22.04 aarch64 VM running on my Mac. I don't think that should make a difference, but I thought I would mention. |
I just cloned the master branch into a fresh directory, built it with
`make -f Makefile static`, then ran `./docker_test.sh` and got all
passing tests.
Just so we make extra sure, can you try the same? (i.e.: clone into a
fresh directory just to make sure the master HEAD is being built &
tested.)
For reference, I'm using an Ubuntu 22.04 aarch64 VM running on my
Mac.
This would be my second suspicion. When you run `docker_test.sh` on a
new machine, it builds a Docker test image named "sslscan-test" with a
couple versions of OpenSSL and GnuTLS; the server executables in this
image are what the locally-built sslscan is tested against. I've only
built this image on x86_64 machines, but perhaps the resulting image is
different on aarch64! Hence, the problem may be with the test image
itself.
What git branch of yours is resulting in these failures? I'll try
running tests on your branch with my x86_64 test image and see what
happens.
|
I spun up an aarch64 machine in AWS and found that some of the tests fail. So this isn't a problem on your end. Interestingly, the tests fail for different reasons in AWS than they do for you! Until this is fixed, I suppose I'll simply run the Docker tests on your PRs as they come in. Please ping me, and I'll do my best to be quick about it! |
I setup an Ubuntu 22.04 host to do docker tests and I'm getting the following errors when run against rbsec/sslscan@master. @jtesta Do you have any insights into what might be going on here?
Thanks.
The text was updated successfully, but these errors were encountered: