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

components/esp-tls/esp_tls.h is missing includes (IDFGH-9392) #10765

Closed
rkit1 opened this issue Feb 14, 2023 · 2 comments
Closed

components/esp-tls/esp_tls.h is missing includes (IDFGH-9392) #10765

rkit1 opened this issue Feb 14, 2023 · 2 comments
Assignees
@AdityaHPatwardhan
Labels
Awaiting Response awaiting a response from the author Status: In Progress Work is in progress Type: Bug bugs in IDF

Comments

@rkit1
Copy link

rkit1 commented Feb 14, 2023

When CONFIG_ESP_TLS_SERVER_SESSION_TICKETS is enabled

C:/.espressif/frameworks/esp-idf-v5.0/components/esp-tls/esp_tls.h:191:5: error: unknown type name 'mbedtls_entropy_context'
  191 |     mbedtls_entropy_context entropy;                                            /*!< mbedTLS entropy context structure */
      |     ^~~~~~~~~~~~~~~~~~~~~~~
C:/.espressif/frameworks/esp-idf-v5.0/components/esp-tls/esp_tls.h:193:5: error: unknown type name 'mbedtls_ctr_drbg_context'
  193 |     mbedtls_ctr_drbg_context ctr_drbg;                                          /*!< mbedTLS ctr drbg context structure.
      |     ^~~~~~~~~~~~~~~~~~~~~~~~
@espressif-bot espressif-bot added the Status: Opened Issue is new label Feb 14, 2023
@github-actions github-actions bot changed the title components/esp-tls/esp_tls.h is missing includes components/esp-tls/esp_tls.h is missing includes (IDFGH-9392) Feb 14, 2023
@AdityaHPatwardhan
Copy link
Collaborator

Hi @rkit1,
Thanks for the issue.
I agree that it is a bug.
I think it was introduced when we restructured the esp_tls.h file some
time back.
Can you please check if the patch that I have attached fixes your issue?
Thanks,
Aditya

server_session_ticket_build_issue.patch.zip

@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Feb 15, 2023
@AdityaHPatwardhan AdityaHPatwardhan added Type: Bug bugs in IDF Awaiting Response awaiting a response from the author labels Feb 16, 2023
@rkit1
Copy link
Author

rkit1 commented Feb 26, 2023

Yes it fixed the issue.

espressif-bot pushed a commit that referenced this issue Mar 12, 2023
@AdityaHPatwardhan @mahavirj
esp-tls: Fix build issue when esp-tls server session tickets have bee…
6692a13 
…n enabled.

Closes #10765
@mahavirj mahavirj mentioned this issue Apr 19, 2023
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AdityaHPatwardhan AdityaHPatwardhan

Labels
Awaiting Response awaiting a response from the author Status: In Progress Work is in progress Type: Bug bugs in IDF
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rkit1 @AdityaHPatwardhan @espressif-bot