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

Bring back NNG logging #242

Merged
merged 22 commits into from
Aug 21, 2024
Merged

Bring back NNG logging #242

merged 22 commits into from
Aug 21, 2024

Conversation

JaylinYu
Copy link
Member

After painful consideration, decided to drop current logging feature of nanosdk, switch to support upstream

gdamore and others added 22 commits August 21, 2024 13:20
The realtime clock is not (yet) exposed for user applications, but it
is used for logging timestamps accurately.

Signed-off-by: jaylin <[email protected]>
This isn't complete, but it should go much further in assisting
debugging TLS related errors.

Signed-off-by: jaylin <[email protected]>
* Add nng_str_sockaddr to get string representation of socket address.

* Added nng_log_get_level() is meant to allow users to obtain the
current level and avoid some possibly expensive operations just
to collect debugging information when debugging is not in effect.

We use a custom logger for NUTS, and this fits within the NUTS
test framework well, so that if -v is supplied we get more content.
All tests now get this by default.

Signed-off-by: jaylin <[email protected]>
Signed-off-by: jaylin <[email protected]>
This allows an efficient way to iterate over the entries
stored in an ID hash. The iteration is fast, and requires no
additional storage.  The order of iteration is not guaranteed.

Signed-off-by: jaylin <[email protected]>
This fixes a problem only found on Windows, that affected both IPC and TCP.
This requires using a supporting version of Mbed TLS.
We have to use PSA crypto for TLS 1.3.

Signed-off-by: jaylin <[email protected]>
Signed-off-by: jaylin <[email protected]>
Signed-off-by: jaylin <[email protected]>
Copy link
Member

@wanghaEMQ wanghaEMQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done

@JaylinYu JaylinYu merged commit 811dea6 into main Aug 21, 2024
13 of 14 checks passed
@JaylinYu JaylinYu deleted the jaylin/nanosdk branch August 21, 2024 14:05
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.

3 participants