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

Add DTLS session resumption tests and fix a bug in SSL ticket key update #1970

Merged

Conversation

hanno-becker
Copy link

Summary: This PR fixes #1968 and #1969.

@simonbutcher
Copy link
Contributor

retest

@simonbutcher
Copy link
Contributor

CI passes. Disregard the 'mbed-ci-generic' result.

@hanno-becker hanno-becker removed the needs-backports Backports are missing or are pending review and approval. label Aug 23, 2018
@hanno-becker hanno-becker requested review from k-stachowiak and removed request for mpg August 29, 2018 14:06
@mpg mpg added needs-backports Backports are missing or are pending review and approval. and removed needs-review Every commit must be reviewed by at least two team members, labels Sep 4, 2018
@mpg
Copy link
Contributor

mpg commented Sep 4, 2018

Re-applying the "needs backports" label as the backports haven't received enough reviews yet (per the label description: "backports are missing or pending review and approval"), so we can see at a glance what is currently blocking this PR from being merged.

@gilles-peskine-arm
Copy link
Contributor

I've provided the second review on the backport. @mpg I think this is now ready for merge.

@gilles-peskine-arm gilles-peskine-arm removed the needs-backports Backports are missing or are pending review and approval. label Sep 4, 2018
@mpg mpg added the approved Design and code approved - may be waiting for CI or backports label Sep 4, 2018
@simonbutcher simonbutcher merged commit 5e863e0 into Mbed-TLS:development Sep 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Design and code approved - may be waiting for CI or backports bug component-tls enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ssl_ticket_update_keys removes keys newer than 1 second
4 participants