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

core: change 'found the format marker' log message from info to debug #283

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

Conversation

mgi388
Copy link

@mgi388 mgi388 commented May 10, 2024

Fixes #272

As in #272, I've also experienced the info log flooding for a message that shouldn't really be info. I know we can look at configuring log levels, but the content of the message seems very much debug-like, not informational.

...
2024-05-10T00:15:25.274616Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
2024-05-10T00:15:25.276794Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
2024-05-10T00:15:25.286236Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
2024-05-10T00:15:25.300764Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
2024-05-10T00:15:25.302995Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
2024-05-10T00:15:25.312544Z  INFO symphonia_core::probe: found the format marker [52, 49, 46, 46] @ 0+2 bytes.
...

Relates to #228.

@mgi388
Copy link
Author

mgi388 commented May 10, 2024

@pdeljanov unlike #228, this one should hopefully be an easy and uncontroversial merge 🙏

@Roms1383
Copy link

Roms1383 commented Sep 2, 2024

+1, my logs are cluttered with the same message

@Roms1383
Copy link

Roms1383 commented Sep 8, 2024

unless someone knows how to filter them out? quickly looking at log crate it does not seem to support fine grained log filtering like let's say env_logger.

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.

Log flood
2 participants