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.

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

TLS-Attacker extended tests and fuzzing #732

Merged
merged 2 commits into from
Nov 20, 2016

Conversation

jurajsomorovsky
Copy link
Contributor

Extended server test suite and policy tests with TLS-Attacker.

TLS-Attacker server fuzzing.

…r direct TLS CBC testing.

CLI TLS server now catches an exception if an invalid connection is received (Otherwise, the server always stopped working)
@codecov-io
Copy link

Current coverage is 88.30% (diff: 100%)

No coverage report found for master at 1e21b64.

Powered by Codecov. Last update 1e21b64...ae2a958

@randombit
Copy link
Owner

Thanks for submitting this. I will merge this now and look at adding the testsuite portion to Travis CI sometime soon, so the tests are run reliably.

From my understanding, with the proper workflows defined TLS-Attacker is also capable of acting as a server which attacks clients which connect to it. It would be great to add tests for that, as well as for DTLS. The system in TLS-Attacker for this looks very easy to use but I have not yet had time to play with it much.

@randombit randombit merged commit ae2a958 into randombit:master Nov 20, 2016
randombit added a commit that referenced this pull request Nov 20, 2016
@jurajsomorovsky
Copy link
Contributor Author

Yes, adding further tests for TLS clients or DTLS functionality should be simple, but is not yet well tested. We are currently refactoring TLS-Attacker and adding new functionalities in the internal branches (e.g., new fuzzing strategies or an action system allowing you to change TLS protocol version in the middle of the protocol flow). Once this is done, we will add also new testing capabilities. However, this can take several weeks.

Thank you for accepting my pull request and integrating TLS-Attacker.

@jurajsomorovsky
Copy link
Contributor Author

Btw., you can find the (probably obvious) travis script I used previously here:
https://github.com/Hackmanit/botan-extended-tests/blob/master/.travis.yml

@jurajsomorovsky jurajsomorovsky deleted the tls-attacker-tests branch December 15, 2016 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants