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

pion/dtls only supports DTLS v1.2 and doesn't properly handle exiting on older versions #74

Closed
voidmoda opened this issue Jun 5, 2019 · 3 comments
Labels
triaged Has been reviewed

Comments

@voidmoda
Copy link

voidmoda commented Jun 5, 2019

Your environment.

  • Version: dtls@v1.3.5
  • Browser: echo show
  • Other Information - I am new to TLS but here's my findings now. According to [RFC5246 7.4.3][https://tools.ietf.org/html/rfc5246#section-7.4.3] it's not legal to send ServerKeyExchange message for RSA while pion/dtls keeps waiting for ServerKeyExchange message in flight3 of client_handlers.go

What did you do?

What did you expect?

What happened?

@Sean-Der
Copy link
Member

Hey @estasig!

Good news, Alexa.RTCSessionController is upgrading to DTLSv1.2!

I haven't gotten any timelines yet, but I was able to get through to the right people. Thank you for reporting this and reaching out on reddit :)

@Sean-Der Sean-Der added the triaged Has been reviewed label Aug 28, 2019
@Sean-Der Sean-Der changed the title DTLS handshake failed with TLS_RSA_WITH_AES_256_CBC_SHA Cipher Suite pion/dtls only supports DTLS v1.2 and doesn't properly handle exiting on older versions Aug 28, 2019
@Sean-Der
Copy link
Member

@estasig I am going to close as a dupe of #3, but please re-open if you disagree.

pion/dtls can be better by actually checking and giving a good error message.

@Sean-Der
Copy link
Member

@estasig This is happening this week! Starting next week you will be able to use Pion with the echo device.

thanks for using Pion and reporting this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged Has been reviewed
Projects
None yet
Development

No branches or pull requests

2 participants