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.

Sign up for GitHub

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

Jump to bottom

Define the API for writing early data on client side in TLS 1.3 #4902

Closed
zhihan opened this issue Aug 31, 2021 · 0 comments · Fixed by #6621
Closed

Define the API for writing early data on client side in TLS 1.3 #4902

zhihan opened this issue Aug 31, 2021 · 0 comments · Fixed by #6621
Assignees
@ronald-cron-arm
Labels
component-tls component-tls13 enhancement size-s Estimated task size: small (~2d)

Comments

@zhihan
Copy link

zhihan commented Aug 31, 2021 •
edited by ronald-cron-arm
Loading

Suggested enhancement

Please see this proposal:
https://docs.google.com/document/d/1h0Uf9ULJA0IgHR2dWhNynatjihQEN5IiFkvmkZD_ikc/edit?usp=sharing

Justification

Mbed TLS needs this to align with other implementations, such as OpenSSL. Also providing a write() method instead of requiring knowing all the early data to be known before then handshake is easier for applications. See the proposal for reasonings.

This issue is created here for visibility. It is referring to this issue in the tls13-prototype fork: hannestschofenig#234

Do we need an mbedtls_ssl_get_early_data_status() alongside mbedtls_ssl_write_early_data() like in the prototype ?
@tom-daubney-arm tom-daubney-arm added api-break This issue/PR breaks the API and must wait for a new major version Community component-tls enhancement component-tls13 labels Aug 31, 2021
@ronald-cron-arm ronald-cron-arm added size-m Estimated task size: medium (~1w) and removed api-break This issue/PR breaks the API and must wait for a new major version labels Sep 22, 2022
@ronald-cron-arm ronald-cron-arm changed the title Change the API for writing early data in TLS 1.3 Define the API for writing early data in TLS 1.3 Sep 22, 2022
@ronald-cron-arm ronald-cron-arm mentioned this issue Sep 22, 2022
Closed
@ronald-cron-arm ronald-cron-arm changed the title Define the API for writing early data in TLS 1.3 Define the API for writing early data on client side in TLS 1.3 Sep 22, 2022
@ronald-cron-arm ronald-cron-arm self-assigned this Sep 22, 2022
@ronald-cron-arm ronald-cron-arm mentioned this issue Sep 22, 2022
Closed
@ronald-cron-arm ronald-cron-arm added size-s Estimated task size: small (~2d) and removed size-m Estimated task size: medium (~1w) labels Sep 22, 2022
This was referenced Sep 23, 2022
Closed
Closed
Closed
@ronald-cron-arm ronald-cron-arm mentioned this issue Nov 9, 2022
Merged
@yuhaoth yuhaoth mentioned this issue Nov 10, 2022
Closed
2 tasks
@ronald-cron-arm ronald-cron-arm mentioned this issue Nov 18, 2022
Merged
3 tasks
@minosgalanakis minosgalanakis moved this to [3.6] TLS 1.3 early data in Past EPICs Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ronald-cron-arm ronald-cron-arm

Labels
component-tls component-tls13 enhancement size-s Estimated task size: small (~2d)
Projects
Past EPICs
Status: [3.6] TLS 1.3 early data
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

TLS 1.3: Add definition of mbedtls_ssl_{write,read}_early_data ronald-cron-arm/mbedtls
5 participants
@daverodgman @zhihan @ronald-cron-arm @bensze01 @tom-daubney-arm