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

Evolution of net_sockets.h in 4.0 #4082

Open
daverodgman opened this issue Jan 27, 2021 · 2 comments
Open

Evolution of net_sockets.h in 4.0 #4082

daverodgman opened this issue Jan 27, 2021 · 2 comments
Labels
api-break This issue/PR breaks the API and must wait for a new major version enhancement size-s Estimated task size: small (~2d)

Comments

@daverodgman
Copy link
Contributor

As a first step, explore this issue and determine the best course of action for Mbed TLS 3.0. Options could include:

  • do nothing
  • move most of the content into a new file outside of the main library

Mailing list discussion: https://lists.trustedfirmware.org/pipermail/mbed-tls/2020-April/000052.html

This is part of #4030

@daverodgman daverodgman added enhancement mbedtls-3 size-s Estimated task size: small (~2d) labels Jan 27, 2021
@mpg
Copy link
Contributor

mpg commented Feb 24, 2021

I don't think there was a strong agreement on the list. The option most likely to eventually get a consensus would be to turn it into a part of a new platform_unix / platform_windows module to be designed.

I think this would require more design work that we can afford for 3.0, and since we agreed to limit 3.0 to simple changes, I'm inclined to post-pone that one to 4.0 and just keep net_sockets as it is for 3.0.

@gilles-peskine-arm
Copy link
Contributor

Given unlimited time, my preference would be to design a new platform module, which would combine things that are today in platform, entropy_poll, timing, net_sockets, etc. We would maintain at least a Unix/POSIX implementation and a Windows implementation of this module — the set of platforms for which we'd maintain an implementation should be the set of platforms we run CI on. OSes that include Mbed TLS and for which our Unix/POSIX implementation doesn't work would maintain their own implementation of this module. This is way out of scope of 3.0.

net_sockets is pretty much what TLS needs. If we move it out of the library, we'd still have to maintain it for the test and sample programs. All we'd gain is that test code needs less robustness. So we lose very little by doing nothing. Doing nothing has my vote.

@mpg mpg added api-break This issue/PR breaks the API and must wait for a new major version and removed mbedtls-3 labels Apr 29, 2021
@gilles-peskine-arm gilles-peskine-arm moved this to Design needed in Mbed TLS 4.0 planning Jun 19, 2024
@gilles-peskine-arm gilles-peskine-arm moved this from Design needed to Requirements needed in Mbed TLS 4.0 planning Jun 19, 2024
@gilles-peskine-arm gilles-peskine-arm changed the title Investigate removing module: net_sockets Evolution of net_sockets.h in 4.0 Aug 14, 2024
@github-project-automation github-project-automation bot moved this to Mbed TLS 4.0 COULD in Backlog for Mbed TLS Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-break This issue/PR breaks the API and must wait for a new major version enhancement size-s Estimated task size: small (~2d)
Projects
Status: Mbed TLS 4.0 COULD
Status: Design needed
Development

No branches or pull requests

4 participants