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

http/tls.lua: Index banned_ciphers by standard name #116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

daurnimator
Copy link
Owner

This alleviates the need for our own standard name to openssl name map for ciphers.
Requires wahern/luaossl#142 and hence a luaossl version requirement bump.

@daurnimator daurnimator force-pushed the banned-ciphers-std-names branch 2 times, most recently from af57f9a to a81d994 Compare July 13, 2018 07:33
@daurnimator
Copy link
Owner Author

daurnimator commented Jul 13, 2018

The cipher TLS_KRB5_WITH_DES_CBC_SHA is missing from the luaossl compat list. See wahern/luaossl#142 (comment)

@daurnimator
Copy link
Owner Author

Raised issue about the 0x001E conflict: openssl/openssl#6710

@daurnimator daurnimator force-pushed the banned-ciphers-std-names branch from a81d994 to 4db1785 Compare August 3, 2018 07:36
@daurnimator daurnimator removed the WIP label Aug 3, 2018
This alleviates the need for our own standard name to openssl name map for ciphers.

Requires a new luaossl release
@daurnimator daurnimator force-pushed the banned-ciphers-std-names branch 2 times, most recently from cbb62f7 to 05f23b1 Compare August 3, 2018 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant