-
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
Make ESXi Plugin work without crypto and fix vm_inventory #697
Conversation
@Matthijsy 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 company="Northwave" |
Co-authored-by: Erik Schamper <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #697 +/- ##
==========================================
- Coverage 74.97% 74.97% -0.01%
==========================================
Files 295 295
Lines 25080 25081 +1
==========================================
Hits 18804 18804
- Misses 6276 6277 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Currently the ESXi module works partly without having PyCryptodome installed, however this package is not 100% needed. Furthermore the vm_inventory method did always return the vm list. This PR does 3 things:
Currently the local layer is created before the mount modules. Some of these modules (in my case vim.v00) can contain empty
vmInventory.xml
files, which results in thevm_inventory
method returning an empty list. By creating this local layer later the version in the local layer is used.