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

partial from pbiering-testing #568

Closed
wants to merge 2 commits into from

Conversation

pbiering
Copy link
Collaborator

  • debug_filter (optional)
  • exception logging (optional)
  • performance logging (optional)
  • minor loglevel adjustments
  • catch some error cases
  • disable announcement of sync-token (still not implemented)
  • example (inactive) configuration for dedicated debug log

pending: will be PRs on-top after migration:

  • "request_token" + some other minor tweaks
  • caching code for items+props (for 2.1 or 3.x)

- debug_filter (optional)
- exception logging (optional)
- performance logging (optional)
- minor loglevel adjustments
- catch some error cases
- disable announcement of sync-token (still not implemented)
@pbiering pbiering mentioned this pull request Mar 12, 2017
@Unrud
Copy link
Collaborator

Unrud commented Jun 4, 2017

I think that most of the stuff in this PR got handled. The debug filtering is overdone in my opinion. Instead I tried to log everything that is important to the user as ERROR or WARNING with useful error messages. Stack traces and other stuff that pollutes the log gets only included when debugging is enabled.

@Unrud
Copy link
Collaborator

Unrud commented Jun 10, 2017

I've added the log file rotation example to the documentation.

@Unrud Unrud closed this Jun 10, 2017
@pbiering pbiering deleted the debug-log-suppress-option branch September 28, 2020 04:53
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