fix: Remove AutoMapper dependency to remediate CVE-2026-32933#1457
fix: Remove AutoMapper dependency to remediate CVE-2026-32933#1457
Conversation
Replace AutoMapper 10.1.1 (and AutoMapper.Extensions.Microsoft.DependencyInjection 8.1.1) with explicit manual property-by-property mapping in ConfigurationMapper.cs. Changes: - Add ConfigurationMapper with explicit MapFrom() for each Args type and ConfigFile - Add type-safe MergeSetting<T>() for config merge with conflict detection - Add explicit ToConfiguration() for InputConfiguration -> Configuration copy - Refactor ConfigPostProcessor to remove IMappingAction interface dependency - Update ConfigurationBuilder to use ConfigurationMapper directly - Update DI registration from AddAutoMapper() to AddTransient<ConfigPostProcessor>() - Remove AutoMapper from all 4 csproj files and Directory.Packages.props - Delete ConfigurationProfile.cs and 9 value converter files No reflection used - every mapped property is listed explicitly for auditability. All 829 unit tests pass. E2E CLI tests verified (generate, validate, redact, aggregate, config merge). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
This PR changes files in the API project. Does it change any of the API interfaces in any way? Please note that this includes the following types of changes:
Because any of these changes can potentially break a downstream consumer with customized interface implementations, these changes need to be treated as breaking changes. Please do one of the following: Option 1 - Publish this as a breaking change
Option 2 - Refactor the changes to be non-breaking
|
That would be good to say in the README too, and in NuGet Gallery. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
This PR changes files in the API project. Does it change any of the API interfaces in any way? Please note that this includes the following types of changes:
Because any of these changes can potentially break a downstream consumer with customized interface implementations, these changes need to be treated as breaking changes. Please do one of the following: Option 1 - Publish this as a breaking change
Option 2 - Refactor the changes to be non-breaking
|
|
/azp run |
Summary
Removes the AutoMapper 10.1.1 NuGet dependency to remediate CVE-2026-32933 (High severity, due 2026-06-12).
Replaces AutoMapper with explicit manual property-by-property mapping in a new \ConfigurationMapper\ class. No reflection is used — every mapped property is listed explicitly for auditability.
Why manual mapping?
Manual mapping is:
What changed
Validation