log2timeline / plaso

Super timeline all the things
https://plaso.readthedocs.io
Apache License 2.0
1.66k stars 327 forks source link

Fix for issue #4787 #4792

Closed jkppr closed 5 months ago

jkppr commented 5 months ago

One line description of pull request

Moving _PrintUserWarning output to stderr and adding a logfile argument to pinfo_tools.

Description:

This fixes the issues described in #4787

Related issue (if applicable): fixes #4787

Notes:

All contributions to Plaso undergo code review. This makes sure that the code has appropriate test coverage and conforms to the Plaso style guide.

One of the maintainers will examine your code, and may request changes. Check off the items below in order, and then a maintainer will review your code.

Checklist:

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4ca7060) 86.02% compared to head (af5c46c) 86.04%. Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4792 +/- ## ========================================== + Coverage 86.02% 86.04% +0.02% ========================================== Files 416 416 Lines 37680 37654 -26 ========================================== - Hits 32414 32400 -14 + Misses 5266 5254 -12 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.