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

Support for DTLS connection IDs #2249

Merged
merged 4 commits into from Apr 1, 2021
Merged

Support for DTLS connection IDs #2249

merged 4 commits into from Apr 1, 2021

Conversation

sergeuz
Copy link
Member

@sergeuz sergeuz commented Dec 8, 2020

Problem

This PR enables the DTLS Connection ID extension (described in this draft RFC).

Steps to Test

Verify that session resumption works as expected.

References

  • [ch55112]

@sergeuz sergeuz marked this pull request as draft December 8, 2020 12:56
@sergeuz sergeuz changed the title Support for DLTS connection IDs Support for DTLS connection IDs Dec 8, 2020
@sergeuz sergeuz force-pushed the dtls_conn_id/ch55112 branch 6 times, most recently from 334422d to c6dbfaf Compare January 28, 2021 15:28
@sergeuz sergeuz added this to the 3.1.0 milestone Jan 28, 2021
@sergeuz sergeuz marked this pull request as ready for review January 28, 2021 19:08
@avtolstoy
Copy link
Member

I rebased and fixed a couple of conflicts around persistent session.

@avtolstoy
Copy link
Member

One thing to consider (as far as I understand DS side of this feature allows) is to remove server ip:port from the persistent storage and perform DNS resolution all the time to allow devices to resume the session on any DS instance. This should allow us to simplify DS deploys, have devices migrate to current DS instances quicker, increase maximum session times and improve connection times for sleepy devices in some cases.

@sergeuz
Copy link
Member Author

sergeuz commented Apr 1, 2021

I rebased and fixed a couple of conflicts around persistent session.

@avtolstoy Thanks!

@sergeuz sergeuz merged commit 6ba482d into develop Apr 1, 2021
@sergeuz sergeuz deleted the dtls_conn_id/ch55112 branch April 1, 2021 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants