Rename debug to ssl_debug + make it internal #6859
Labels
api-break
This issue/PR breaks the API and must wait for a new major version
deprecation
needs deprecation announcement
enhancement
size-s
Estimated task size: small (~2d)
The debug module, currently
debug.h
+debug.c
is not about generic debugging but actually part of SSL. It's the only part of SSL that doesn't start withssl_
, so this should be fixed by renaming tossl_debug
. (Once this is done, all parts oflibmbedtls.a
will start withssl_
and all parts oflibmbedx509.a
withx509_
, so once crypto is split out everything will be neat and tidy.)Also, it's unclear is most of whats currently in
debug.h
is really meant as a public API for applications to use, or only for internal use by the library. IMO onlymbedtls_debug_set_threshold()
should really be public and everything else internal. If we go this way, then we might want to move its declaration tossl.h
(and rename tombedtls_ssl_debug_set_threshold()
while at it), and then have no publicssl_debug.h
, just an internal one inlibrary
.Similarly
MBEDTLS_DEBUG_C
should be renamedMBEDTLS_SSL_DEBUG_C
for consistency.The text was updated successfully, but these errors were encountered: