Skip to content

Remove mbedtls_pk_type_t from the API #210

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

Open
gilles-peskine-arm opened this issue Mar 14, 2025 · 0 comments · May be fixed by #204
Open

Remove mbedtls_pk_type_t from the API #210

gilles-peskine-arm opened this issue Mar 14, 2025 · 0 comments · May be fixed by #204
Assignees
Labels
api-break This issue/PR breaks the API and must wait for a new major version size-s Estimated task size: small (~2d)

Comments

@gilles-peskine-arm
Copy link
Contributor

gilles-peskine-arm commented Mar 14, 2025

Move the parts of pk.h that are becoming private in TF-PSA-Crypto out of pk.h and ensure that they are no longer documented as public APIs. This notably includes mbedtls_pk_type_t. Adjust PK and consuming APIs in Mbed TLS accordingly.

See #203 for the architectural specification, including the list of all the API elements made private, and how to make mbedtls_pk_type_t private.

Removing the use of mbedtls_pk_type_t in library code, test code and sample programs is out of scope and will likely not be done until after TF-PSA-Crypto 1.0 is released.

Where there are changes to the specification of existing functions, unit tests for the new behavior is out of scope of this issue, and will be done in a follow-up #207.

@gilles-peskine-arm gilles-peskine-arm added api-break This issue/PR breaks the API and must wait for a new major version size-s Estimated task size: small (~2d) labels Mar 14, 2025
@gilles-peskine-arm gilles-peskine-arm self-assigned this Mar 14, 2025
@gilles-peskine-arm gilles-peskine-arm moved this to Implementation in progress in Mbed TLS 4.0 planning Mar 14, 2025
@gilles-peskine-arm gilles-peskine-arm linked a pull request Mar 14, 2025 that will close this issue
5 tasks
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 size-s Estimated task size: small (~2d)
Projects
Status: Implementation in progress
Status: PK API for 1.0
Development

Successfully merging a pull request may close this issue.

1 participant