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

Fix tls_hooks18.test.py records.config line. #7643

Closed
wants to merge 1 commit into from

Conversation

bneradt
Copy link
Contributor

@bneradt bneradt commented Mar 29, 2021

This fixes the tests/gold_tests/tls_hooks/tls_hooks18.test.py that
accidentally had an extra line left in from the cleanup commit:

8f25b6b

All this actually does is remove:

'ECDHE-RSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-RSA-AES256-SHA384:AES128-GCM-SHA256:AES256-GCM-SHA384:ECDHE-RSA-RC4-SHA:ECDHE-RSA-AES128-SHA:ECDHE-RSA-AES256-SHA:RC4-SHA:RC4-MD5:AES128-SHA:AES256-SHA:DES-CBC3-SHA!SRP:!DSS:!PSK:!aNULL:!eNULL:!SSLv2', })

The rest of this is just autopep8 reformatting the file.

This fixes the tests/gold_tests/tls_hooks/tls_hooks18.test.py that
accidentally had an extra line left in from the cleanup commit:

8f25b6b
@bneradt bneradt removed this from the 10.0.0 milestone Mar 29, 2021
@bneradt
Copy link
Contributor Author

bneradt commented Mar 29, 2021

Closing this to add a second fix for tls_client_versions.test.py

@bneradt bneradt closed this Mar 29, 2021
@bneradt bneradt deleted the fix_tls_hook18_file branch June 9, 2021 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants