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

Remove use of mbedtls_cipher_info from ssl_context_info.c #9977

Open
davidhorstmann-arm opened this issue Feb 13, 2025 · 0 comments
Open

Remove use of mbedtls_cipher_info from ssl_context_info.c #9977

davidhorstmann-arm opened this issue Feb 13, 2025 · 0 comments
Labels
size-s Estimated task size: small (~2d)

Comments

@davidhorstmann-arm
Copy link
Contributor

Part of implementing #9904

Modify ssl_context_info.c to remove the use of mbedtls_cipher_ functions (these are used to get the name of the cipher being used).

Either:

  • Find out how to replace this with equivalent code that gets the name of the ciphersuite in a way that does not use the legacy API, or
  • Remove the pretty-printing of the cipher name and simply print a numerical value that represents the cipher being used. This already happens when MBEDTLS_CIPHER_C is not configured, but it needs some investigation to check that this is still the same when we are using PSA.

This task is done when ssl_context_info.c does not use any mbedtls_cipher functions (or for that matter any legacy crypto functions).

Estimate: 4 Ideal Engineering Hours

@davidhorstmann-arm davidhorstmann-arm added the size-s Estimated task size: small (~2d) label Feb 13, 2025
@davidhorstmann-arm davidhorstmann-arm moved this to Implementation needed in Mbed TLS 4.0 planning Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-s Estimated task size: small (~2d)
Projects
Status: Implementation needed
Development

No branches or pull requests

1 participant