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 1.3 and Cipher-suite #1947

Open
Ys88 opened this issue Jan 20, 2019 · 5 comments
Open

TLS 1.3 and Cipher-suite #1947

Ys88 opened this issue Jan 20, 2019 · 5 comments

Comments

@Ys88
Copy link

Ys88 commented Jan 20, 2019

Hi, and Happy New Year by the same occasion,

I was wondering if this is possible to select the preferred order of the ciphers, when using TLS v1.3.

I am using:

  • the latest H2O sources,
  • OpenSSL 1.1.1,
  • "cipher-preference: server".

However, no matter what I put in the "cipher-suite", this is always the default OpenSSL suite, which is used : "TLS_AES_256_GCM_SHA384", "TLS_CHACHA20_POLY1305_SHA256" and "TLS_AES_128_GCM_SHA256" and in this exact order.

So I was wondering if this is the expected behavior?

Thank you,

Best regards,

@deweerdt
Copy link
Member

The cipher list is currently hardcoded. My first instinct was to extract the cipher list from the ssl_ctx but the way to specify ciphers has changed in openssl 1.1.1, so we need to call SSL_CTX_set_ciphersuites

I'm not sure what the best way to tackle that would be, a separate configuration option cipher-suite-tls1.3 comes to mind, what do you think @kazuho ?

@robguima
Copy link
Collaborator

Also, if the underlying openssl is older than openssl-1.1.1 (say openssl-1.1.0), the ssl_ctx will not contain the TLS1.3 ciphers. A separate list like @deweerdt suggested could make the usage simpler.

@Ys88
Copy link
Author

Ys88 commented Feb 2, 2019

A separate configuration option has my vote.

I believe it would be easier and clearer. Mixing ciphers for TLSv1.3 with those of the previous TLS versions, in the same option, would make it harder to read, and certainly harder to parse (I guess).

@deweerdt
Copy link
Member

deweerdt commented Feb 2, 2019

I've pushed a proposal to #1963 , please let me know what you think.

@Ys88
Copy link
Author

Ys88 commented Feb 3, 2019

I do not have the skill to review the code itself, but the idea seems good to me.

Is there a naming convention for options in H2O ? I do not mind, but can it cause confusion to have a "dot" inside an option name? "cipher-suite-tls1.3" or "cipher-suite-tls13" ? As I said, I do not mind, this was just a thought which came to my mind.

Thanks!

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

No branches or pull requests

3 participants