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

mbedTLS module does not have CONFIG option for HKDF #59984

Open
asousa-zg opened this issue Jul 4, 2023 · 5 comments
Open

mbedTLS module does not have CONFIG option for HKDF #59984

asousa-zg opened this issue Jul 4, 2023 · 5 comments
Assignees
Labels
area: Crypto / RNG Enhancement Changes/Updates/Additions to existing features

Comments

@asousa-zg
Copy link

Describe the bug
I want to enable mbedTLS HKDF but zephyr module does not have a CONFIG option to enable this.
It is possible to add this one?

The file zephyr/modules/mbedtls/configs/config-tls-generic.h have configs for almost every mbedtls libraries but not for HKDF.
The solution would be to add to zephyr/modules/mbedtls/Kconfig.tls-generic a CONFIG for HKDF.

At this moment, the current solution for me is to define my own MBEDTLS_CFG_FILE, but it would be better to have it in the zephyr.

To Reproduce
Steps to reproduce the behavior:

  1. Include in one file the mbedtls_hkdf() function
  2. west build
  3. See error: undefined reference to `mbedtls_hkdf'
@asousa-zg asousa-zg added the bug The issue is a bug, or the PR is fixing a bug label Jul 4, 2023
@github-actions
Copy link

github-actions bot commented Jul 4, 2023

Hi @asousa-trd! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

@henrikbrixandersen
Copy link
Member

This sounds more like an enhancement request than a bug to me?

@asousa-zg
Copy link
Author

This sounds more like an enhancement request than a bug to me?

Yes, you're right. Should I close and open it in enhancement?

@nordicjm nordicjm added Enhancement Changes/Updates/Additions to existing features and removed bug The issue is a bug, or the PR is fixing a bug labels Jul 4, 2023
@henrikbrixandersen
Copy link
Member

Yes, you're right. Should I close and open it in enhancement?

No need. It was reclassified as an enhancement.

@zephyrbot
Copy link
Collaborator

Hi @d3zd3z, @ceolin,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. It was just assigned to you based on the labels. If you don't consider yourself the right person to address this issue, please re-assing it to the right person.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

@asousa-zg you are also encouraged to help moving this issue forward by providing additional information and confirming this request/issue is still relevant to you.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Crypto / RNG Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

6 participants