fox-it / flow.record

Recordization library
GNU Affero General Public License v3.0
7 stars 9 forks source link

Use reprlib to limit the warning message #101

Closed yunzheng closed 8 months ago

yunzheng commented 8 months ago

The warning message could get very long (single line) due to extra data in msgpack parsing. For example due to corrupt file, or incorrect use.

By using reprlib we limit the length of the warning message.

codecov[bot] commented 8 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (676d61c) 80.06% compared to head (d6d6069) 80.08%.

Files Patch % Lines
flow/record/stream.py 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #101 +/- ## ========================================== + Coverage 80.06% 80.08% +0.01% ========================================== Files 33 33 Lines 3110 3113 +3 ========================================== + Hits 2490 2493 +3 Misses 620 620 ``` | [Flag](https://app.codecov.io/gh/fox-it/flow.record/pull/101/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=fox-it) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/fox-it/flow.record/pull/101/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=fox-it) | `80.08% <75.00%> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=fox-it#carryforward-flags-in-the-pull-request-comment) to find out more.

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