rabbitmq / rabbitmq-common

Common library used by rabbitmq-server and rabbitmq-erlang-client
Other
66 stars 112 forks source link

Remove TLSv1.3 from excluded TLS versions #416

Closed nicocaille closed 4 years ago

nicocaille commented 4 years ago

As OTP23 is released and is supporting TLSv1.3 by default, removed TLSv1.3 from list of bad SSL protocols.

Closes #415.

pivotal-issuemaster commented 4 years ago

@nicocaille Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

pivotal-issuemaster commented 4 years ago

@nicocaille Thank you for signing the Contributor License Agreement!

michaelklishin commented 4 years ago

We still only have so much experience with TLSv1.3 support in Erlang but if OTP 23 claims to support it, we should let people try it out for themselves.

michaelklishin commented 4 years ago

Backported to v3.8.x.

gerhard commented 4 years ago

Nice one @nicocaille!

What are your thoughts @michaelklishin on adding tls 1.0 & tls1.1 to this list?

image

michaelklishin commented 4 years ago

Sounds good to me.