-
Notifications
You must be signed in to change notification settings - Fork 142
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-table doesn't seem to be working #292
Comments
I've commented out the reference for now in #296 and made all the necessary updates manually. In the future I'll look into what changes are needed to parse the new C header file definition formats in |
Would you please be more specific about what does not seem to work? Then, I can take a closer look, and possible choose different data sources.
https://pagure.io/mod_nss/blob/master/f/nss_engine_cipher.c contains a mapping from NSS cipher name to OpenSSL cipher name. |
@janbrasna please test with april/tls-table#10 which integrates the other PRs and issues April's repo. If desirable to use different data sources as I proposed in comment above, then please file a separate issue request. |
Mainly the Besides that some of the mapping seems weird (0x00, 0x0A missing its OpenSSL name; 0x00, 0x12 missing both IANA and OpenSSL, but they are listed in the sources used.) (Testing the fork at
It's supposed to be run locally, so the reason to scrape the latest source is probably to avoid any version- or OS-specific data the binaries return, or filters used to rule out any ciphers not supported (by default or at all) anymore. I believe there's more codepoints in the definitions than effectively enabled at any level for the latest versions. (NB: you'd need to depend on openssl and gnutls packages just to run the cli commands, i.e. none of the systems I work on would have gnutls-cli available, and system default openssl is actually a LibreSSL version — so this would need to build recent versions just to list the ciphers.) |
This tool
https://github.com/april/tls-table/blob/master/tls-table.py
is used to produce the Cipher Suites mediawiki page but it doesn't seem to work. We should either get it working or remove reference to it if we won't use it going forward.
The text was updated successfully, but these errors were encountered: