Visitar URL original
[6.1.8] Backport dependency servicing and synchronize nuspecs by mdaigle · Pull Request #4870 · dotnet/SqlClient · GitHub
Skip to content

[6.1.8] Backport dependency servicing and synchronize nuspecs - #4870

Merged
mdaigle merged 1 commit into
release/6.1from
dev/mdaigle/backport-4842-6-1-8
Oct 22, 2026
Merged

mdaigle merged 1 commit into
release/6.1from
dev/mdaigle/backport-4842-6-1-8

Conversation

@mdaigle

@mdaigle mdaigle commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Description

Manually backport the applicable dependency servicing updates from #4842 together with the nuspec synchronization from #4862 for 6.1.8. The original PR came from a fork, so the backport action did not run.

  • Update existing .NET 9 references to Microsoft.Extensions.Caching.Memory, System.Configuration.ConfigurationManager, and System.Security.Cryptography.Pkcs from 9.0.11 to 9.0.21 in central package versions and the driver nuspec.
  • Update System.Security.Cryptography.Pkcs from 8.0.1 to 8.0.2 for .NET Framework 4.6.2, .NET 8, and .NET Standard 2.0.
  • Synchronize the AKV provider's .NET 9 Microsoft.Extensions.Caching.Memory dependency to 9.0.21, matching the shared central props it imports.

Scope is exactly three files and 13 version-attribute changes. Preserve the release/6.1 framework matrix, dependency exclusions, and existing dependency set; do not add dependencies absent from this branch. System.Diagnostics.DiagnosticSource and unrelated XML content remain unchanged. No public API, runtime implementation, documentation, localization, or pipeline changes are included; no breaking API changes are introduced.

Issues

Backports #4842 and includes the corresponding nuspec alignment from #4862 for milestone 6.1.8.

Testing

The implementation was validated before this publication-only handoff using .NET SDK 9.0.312. This PR publishes the same validated commit, 0902e5296f118e73ad82e4260f8fe913da836dc6; the builds were not repeated during publication.

Command Result
dotnet msbuild build.proj -t:BuildAll -p:Configuration=Release -p:BuildForRelease=false -verbosity:minimal Passed: driver build.
dotnet build src\Microsoft.Data.SqlClient\add-ons\AzureKeyVaultProvider\Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider.csproj -c Release -p:BuildProjectReferences=false -verbosity:minimal Passed: AKV provider build, 0 warnings and 0 errors.
dotnet msbuild build.proj '-t:BuildTools;BuildNetCoreAllOS;GenerateNugetPackage' -p:Configuration=Release -p:BuildForRelease=false -verbosity:minimal Passed: tools, Windows/Unix/AnyOS driver assets, and driver NuGet package generation.
git diff --check Passed during implementation.
git diff --check origin/release/6.1 HEAD Passed again during publication.

Dependency validation established that the baseline contains seven unserviced nuspec entries, all 65 literal dependency declarations across seven driver/AKV framework groups match evaluated central versions after the change, exactly 13 intended version attributes changed, and all 56 dependencies in the generated driver nupkg match the source nuspec. Publication reran & 'C:\Users\mdaigle\.copilot\session-state\aceee497-f1fa-43a0-a0f0-d1c7ed3906bf\files\Validate-BackportDependencies.ps1': passed the 65-dependency alignment, seven servicing entries, and 13-attribute-only checks. This validation script is a local session artifact, not a repository change.

Limitations: unit/integration tests and AKV package generation were not run. This is a dependency-metadata-only backport with no runtime code changes; validation focused on dependency evaluation, driver/AKV builds, and the generated driver package. BuildTools/.NET 8 runtime prerequisites and SDK installation failures were resolved without repository changes. The generated driver package retains the branch's unchanged 6.1.7-dev version; this PR does not change release versioning.

  • Tests added or updated (none; dependency metadata checks and existing builds used instead).
  • Public API changes documented (not applicable; no public API changes).
  • Verified against customer repro (not applicable; no customer repro).
  • Ensure no breaking changes introduced (framework matrix, API surface, and dependency set preserved).

Guidelines

Please review the contribution guidelines before submitting a pull request:

Backport applicable dependency updates from #4842 and nuspec synchronization from #4862 for Hotfix 6.1.8. Preserve the release/6.1 framework matrix and unused-package exclusions.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: aceee497-f1fa-43a0-a0f0-d1c7ed3906bf
Copilot AI balanced review requested due to automatic review settings October 19, 2026 18:05
@mdaigle
mdaigle requested a review from a team as a code owner October 19, 2026 18:05
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Oct 19, 2026
@mdaigle mdaigle added this to the 6.1.8 milestone Oct 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The 13 version updates match the intended servicing scope, remain aligned across package metadata, and have no identified blocking issues.

No open issues 🎉

What changed in this PR

Backports dependency servicing for 6.1.8 and aligns published package requirements with central package versions.

Changes:

  • Updates selected .NET 9 dependencies to 9.0.21 and other targeted PKCS dependencies to 8.0.2.
  • Synchronizes driver and Azure Key Vault provider nuspecs while preserving frameworks, dependency sets, and exclusions.
File Description
tools/​specs/​Microsoft.Data.SqlClient.nuspec Raises driver dependency minimums to serviced versions.
tools/​specs/​add-ons/​Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider.nuspec Aligns the .NET 9 caching dependency.
src/​Directory.Packages.props Updates corresponding central package versions.

Review effort: Balanced

@mdaigle mdaigle linked an issue Oct 19, 2026 that may be closed by this pull request
@mdaigle
mdaigle enabled auto-merge (squash) October 19, 2026 18:16
@mdaigle

mdaigle commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).
3 pipeline(s) were filtered out due to trigger conditions.

This was referenced Nov 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[6.1] Backport #4843

7 participants