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

esp-tls.c and CONFIG_MBEDTLS_DEBUG_LEVEL (IDFGH-1219) #3521

Closed
WallaceWilliam opened this issue May 22, 2019 · 3 comments
Closed

esp-tls.c and CONFIG_MBEDTLS_DEBUG_LEVEL (IDFGH-1219) #3521

WallaceWilliam opened this issue May 22, 2019 · 3 comments

Comments

@WallaceWilliam
Copy link

please, replace in esp-tls.c line 331

#ifdef CONFIG_MBEDTLS_DEBUG
    mbedtls_esp_enable_debug_log(&tls->conf, 4);
#endif

to

#ifdef CONFIG_MBEDTLS_DEBUG
    mbedtls_esp_enable_debug_log(&tls->conf, CONFIG_MBEDTLS_DEBUG_LEVEL);
#endif
@github-actions github-actions bot changed the title esp-tls.c and CONFIG_MBEDTLS_DEBUG_LEVEL esp-tls.c and CONFIG_MBEDTLS_DEBUG_LEVEL (IDFGH-1219) May 22, 2019
@mahavirj
Copy link
Member

@WallaceWilliam Any specific reason why you are recommending this? I do not see MBEDTLS_DEBUG_LEVEL as config option either.

@negativekelvin
Copy link
Contributor

I think he is requesting that the debug threshold be configurable and that includes adding the config option.

@WallaceWilliam
Copy link
Author

I think he is requesting that the debug threshold be configurable and that includes adding the config option.

so much
if threshold = 4 a lot of debug information is issued

@igrr igrr closed this as completed in 0a5dfd3 Jun 10, 2019
trombik pushed a commit to trombik/esp-idf that referenced this issue Aug 9, 2019
0xFEEDC0DE64 pushed a commit to 0xFEEDC0DE64/esp-idf that referenced this issue May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants