-
Notifications
You must be signed in to change notification settings - Fork 47
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
Fix parsing journal when value is string None #746
Conversation
A journal entry can contain fields with the string value None. Currently parsing to int fails as it only checks for None type and not None string.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #746 +/- ##
=======================================
Coverage 75.05% 75.05%
=======================================
Files 295 295
Lines 25084 25084
=======================================
Hits 18827 18827
Misses 6257 6257
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
The added comment explains previous change of also checking for stringy None in journal parsing Co-authored-by: cecinestpasunepipe <[email protected]>
Could you still apply the function to the relevant fields in the |
@h0ckeyst1ck thank you for your contribution! As this is your first code contribution, please read the following Contributor License Agreement (CLA). If you agree with the CLA, please reply with the following information:
Contributor License Agreement
Contribution License AgreementThis Contribution License Agreement ("Agreement") governs your Contribution(s) (as defined below) and conveys certain license rights to Fox-IT B.V. ("Fox-IT") for your Contribution(s) to Fox-IT"s open source Dissect project. This Agreement covers any and all Contributions that you ("You" or "Your"), now or in the future, Submit (as defined below) to this project. This Agreement is between Fox-IT B.V. and You and takes effect when you click an “I Accept” button, check box presented with these terms, otherwise accept these terms or, if earlier, when You Submit a Contribution.
|
Hi Miauwkeru! |
@DissectBot agree |
The current change only defines the function |
@h0ckeyst1ck we moved this pr to #944 and implemented the fixes ourselves, thanks for your contribution! |
A journal entry can contain fields with the string value None. Currently parsing to int fails as it only checks for None type and not None string.
fixes #736