-
Notifications
You must be signed in to change notification settings - Fork 24
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
fix(logging): improve node core's debug logging #286
Conversation
c7621cd
to
f7a435d
Compare
f7a435d
to
b700d53
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I really like the request-wrapper related changes. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
3d73293
to
16768c4
Compare
This commit includes the following changes: 1. some additional debug logging in various components 2. HTTP message tracing was improved by adding specific interceptors to display request/response messages much the same way the Go and Java cores do. Signed-off-by: Phil Adams <phil_adams@us.ibm.com>
16768c4
to
3e31202
Compare
## [5.0.2](v5.0.1...v5.0.2) (2024-09-03) ### Bug Fixes * **logging:** improve node core's debug logging ([#286](#286)) ([7bcb404](7bcb404))
🎉 This PR is included in version 5.0.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
This commit adds some debug logging in various components.
Edit: it also includes a more custom logger for http request/response messages