Skip to content

Add medium-impact breaking change note for Microsoft.Data.SqlClient 7.0 upgrade#5305

Merged
AndriySvyryd merged 3 commits intopreview-3from
copilot/add-breaking-change-note
Mar 26, 2026
Merged

Add medium-impact breaking change note for Microsoft.Data.SqlClient 7.0 upgrade#5305
AndriySvyryd merged 3 commits intopreview-3from
copilot/add-breaking-change-note

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 26, 2026

Documents the breaking change from dotnet/efcore#37949 which upgrades Microsoft.Data.SqlClient from 6.x to 7.0 in EF Core 11.

SqlClient 7.0 removes Azure/Entra ID authentication dependencies from the core package. Apps using Entra ID auth will fail at runtime unless they add the new extension package:

<PackageReference Include="Microsoft.Data.SqlClient.Extensions.Azure" Version="7.0.0" />

Changes to entity-framework/core/what-is-new/ef-core-11.0/breaking-changes.md:

  • Added entry to the summary table as Medium impact
  • Added full breaking change section under Medium-impact changes covering old/new behavior, rationale, and mitigation
  • Notes ActiveDirectoryPassword obsolescence and links to SqlClient 7.0 release notes

⌨️ Start Copilot coding agent tasks without leaving your editor — available in VS Code, Visual Studio, JetBrains IDEs and Eclipse.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation to EF Core 11 breaking changes describing the impact of upgrading Microsoft.Data.SqlClient to 7.0, particularly the need to add the Azure/Entra ID extensions package for Entra authentication scenarios.

Changes:

  • Adds a new Low-impact entry to the EF Core 11 breaking changes summary table.
  • Adds a full breaking-change section for the SqlClient 7.0 upgrade (old/new behavior, why, mitigations), including the required Microsoft.Data.SqlClient.Extensions.Azure package reference.

Copilot AI requested a review from AndriySvyryd March 26, 2026 01:20
@AndriySvyryd AndriySvyryd marked this pull request as ready for review March 26, 2026 07:20
Copilot AI review requested due to automatic review settings March 26, 2026 07:20
@AndriySvyryd AndriySvyryd assigned roji and unassigned AndriySvyryd and Copilot Mar 26, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copy link
Copy Markdown
Member

@roji roji left a comment

Choose a reason for hiding this comment

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

I'd bump the impact from low to medium, since it's likely to affect a great deal of users (though the mitigation is trivial).

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 26, 2026

I'd bump the impact from low to medium, since it's likely to affect a great deal of users (though the mitigation is trivial).

Done in aa58379 — bumped to Medium impact in both the summary table and moved the full section under ## Medium-impact changes.

Copilot AI changed the title Add low-impact breaking change note for Microsoft.Data.SqlClient 7.0 upgrade Add medium-impact breaking change note for Microsoft.Data.SqlClient 7.0 upgrade Mar 26, 2026
Copilot AI requested a review from roji March 26, 2026 08:52
Copilot finished work on behalf of roji March 26, 2026 08:52
@roji roji assigned roji and unassigned AndriySvyryd Mar 26, 2026
@roji
Copy link
Copy Markdown
Member

roji commented Mar 26, 2026

@AndriySvyryd note that copilot went ahead and implemented some changes I suggested.

@roji roji assigned AndriySvyryd and unassigned roji Mar 26, 2026
@ErikEJ
Copy link
Copy Markdown
Contributor

ErikEJ commented Mar 26, 2026

LGTM

Is the general policy to depend only on SqlClient LTS releases in any EF Core version?

@roji
Copy link
Copy Markdown
Member

roji commented Mar 26, 2026

I think we're OK with depending on STS SqlClient from STS EF versions, but @AndriySvyryd knows best.

@AndriySvyryd AndriySvyryd merged commit 90d95b7 into preview-3 Mar 26, 2026
5 checks passed
@AndriySvyryd AndriySvyryd deleted the copilot/add-breaking-change-note branch March 26, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants