-
Notifications
You must be signed in to change notification settings - Fork 96
feat: #560 disable console log #590
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
Conversation
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.
I have a question regarding the tracing configuration: Is it possible to selectively disable log messages originating from the WriteApi? For instance, messages like the ones shown in the screenshot below can be quite distracting:
However, other messages may still be valuable for our customers. The customer looking for a way to suppress only the WriteApi
messages without affecting the rest. Is it possible?
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.
Thank you for your PR 👍. We're nearly there. Could you please rename the CategoryTraceFilter.cs
file to InfluxDBTraceFilter.cs
and address the latest minor requirements:
392751f
to
3d82f47
Compare
3d82f47
to
6343308
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.
LGTM 🚀
Closes #560
Proposed Changes
Allows disable Trace verbose messages
Checklist
dotnet test
completes successfully