-
Notifications
You must be signed in to change notification settings - Fork 54
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
Get scheduled task display name with filename #401
Get scheduled task display name with filename #401
Conversation
If Principals/Principal/DisplayName is empty, use the filename as the display_name
@0x49736b 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.
|
@DissectBot agree |
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
Could you adjust the unit tests with the newly filled in field? |
Co-authored-by: Erik Schamper <1254028+Schamper@users.noreply.github.com>
Codecov Report
@@ Coverage Diff @@
## main #401 +/- ##
=======================================
Coverage 73.28% 73.28%
=======================================
Files 250 250
Lines 19994 19994
=======================================
Hits 14653 14653
Misses 5341 5341
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
If Principals/Principal/DisplayName is empty, use the filename as the display_name.
I noticed that the field "Principals/Principal/DisplayName" is not used a lot, resulting in the Display Name being null for most of the records.
Instead, let's use the filename as Display Name (Like in the Task Scheduler GUI)