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

coap: zephyr: add log level zephyr coap client #612

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

szczys
Copy link
Contributor

@szczys szczys commented Sep 20, 2024

The coap_client_zephyr.c file registers for logging but it does not pass a log level. This meant that enabling debug logging for the Golioth Firmware SDK did not include enabling logs from the zephyr client. This commit adds GOLIOTH_DEBUG_DEFAULT_LOG_LEVEL as the default log level.

resolves https://github.com/golioth/firmware-issue-tracker/issues/598

The coap_client_zephyr.c file registers for logging but it does not pass a
log level. This meant that enabling debug logging for the Golioth Firmware
SDK did not include enabling logs from the zephyr client. This commit adds
GOLIOTH_DEBUG_DEFAULT_LOG_LEVEL as the default log level.

Signed-off-by: Mike Szczys <mike@golioth.io>
Copy link

Visit the preview URL for this PR (updated for commit ecbb9a1):

https://golioth-firmware-sdk-doxygen-dev--pr612-szczys-coap-ze-7mcyszne.web.app

(expires Fri, 27 Sep 2024 00:06:21 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

Code Coverage (Linux)

Type Coverage
lines 65.6% (1402 of 2138 lines)
functions 78.1% (139 of 178 functions)

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

Successfully merging this pull request may close these issues.

2 participants