2.0.0.569 #36
Merged
2.0.0.569 #36
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
adamcongdon
commented
Dec 16, 2024
adamcongdon
commented
- 12.3 values
- Semgrep scanning
- Added `System.Management.Automation` and `Microsoft.Management.Infrastructure` namespaces for PowerShell script execution. - Introduced `CScripts` class in `CScripts.cs` for encapsulating PowerShell script execution logic, including methods for retrieving Veeam job information. - Added new methods to `PSInvoker.cs` for invoking embedded PowerShell scripts, specifically for VB365 data collection. - Enhanced `CCsvParser.cs` with methods for dynamic retrieval of Entra ID tenant and job data. - Updated `CProtectedWorkloads` class to include a new list for Entra workloads. - Improved `CHtmlTables.cs` with new HTML table generation logic for Entra ID tenant and job data. - Updated `CMalwareObject.cs` with additional properties related to malware detection and notification options. - Enhanced `Get-VBRConfig.ps1` script to include logic for collecting Entra ID tenant and job data, error handling, and exporting data to CSV files. - Updated project files (`VeeamHealthCheck.csproj` and `VhcXTests.csproj`) to include new package references for PowerShell and other dependencies, and updated assembly and file versions. - Made minor formatting and consistency improvements across various files, including adding missing semicolons and proper indentation. - Added embedded resources to include PowerShell scripts as part of the build output. - Adjusted memory task calculations for VBR version 12 in `CProxyDataFormer.cs`. - Included `Microsoft.CodeAnalysis` namespace in `CHtmlCompiler.cs`. - Improved formatting and alignment of HTML table data in `CHtmlTables.cs`. These changes introduce new functionality for collecting and reporting Entra ID tenant and job data, enhance PowerShell script execution capabilities, and improve the overall maintainability and reliability of the application.
Refactor NAS handling and update version - Removed `NFSVersion` property from `CNasVmcInfo` and adjusted `Index` attributes. - Refactored NAS HTML table logic in `CHtmlTables` to conditionally display based on NAS workload presence. - Updated `NasSourceInfo` with additional namespaces, improved NAS workload logic, and enhanced error handling in `CalculateStorageString`. - Changed remote debug machine IP in `launchSettings.json`. - Removed NAS HTML table creation function from `ReportScript.js`.
…ltiple redundant pieces of code..
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.