Expose SSPI context provider as public

[twsouthwick]

This change:

• Adds a property to SqlConnection to allow setting a provider
• Plumbs that property into the TdsParser so that it can be used if set

Fixes #2253

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (28caf4c) to head (05d4a8d).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #2494       +/-   ##
==========================================
- Coverage   90.82%       0   -90.83%     
==========================================
  Files           6       0        -6     
  Lines         316       0      -316     
==========================================
- Hits          287       0      -287     
+ Misses         29       0       -29     

Flag	Coverage Δ
addons	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[benrr101]

Overall, really like the goal and implementation. Most of my comments are style related, since I'm sure my colleagues have tackled the bigger questions.

[paulmedynski]

Made an initial pass, and came up with some questions/comments.

[paulmedynski]

Factory vs instance in the netfx code, and a few questions about docs.

[nhart12]

Latest package still works for our use-case

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlConnection.cs:807

• The SspiContextProvider property setter is missing critical validation that exists in other similar properties (Credential, AccessToken, AccessTokenCallback). Since this property affects the connection pool key, it must validate that the connection is not already open or opening before allowing the value to be changed.

Add the following validation at the start of the setter:

if (!InnerConnection.AllowSetConnectionString)
{
    throw ADP.OpenConnectionPropertySet(nameof(SspiContextProvider), InnerConnection.State);
}

This prevents runtime errors when users attempt to change the SSPI provider on an active connection and maintains consistency with the established pattern for pool-key-affecting properties.

        public SspiContextProvider SspiContextProvider
        {
            get { return _sspiContextProvider; }
            set
            {
                ConnectionString_Set(new SqlConnectionPoolKey(_connectionString, credential: _credential, accessToken: null, accessTokenCallback: null, sspiContextProvider: value));
                _sspiContextProvider = value;
            }
        }

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlConnection.cs:806

• The SspiContextProvider property setter is missing validation that prevents modification after the connection is opened. Connection-level properties that affect the pool key (Credential, AccessToken, AccessTokenCallback) must validate InnerConnection.AllowSetConnectionString and throw ADP.OpenConnectionPropertySet if false. The documentation at line 2145 explicitly states "The SspiContextProvider is a part of the connection pool key", so it must follow the same pattern. Add the validation check at the start of the setter, similar to the AccessTokenCallback property at lines 782-784.

        public SspiContextProvider SspiContextProvider
        {
            get { return _sspiContextProvider; }
            set
            {
                ConnectionString_Set(new SqlConnectionPoolKey(_connectionString, credential: _credential, accessToken: null, accessTokenCallback: null, sspiContextProvider: value));
                _sspiContextProvider = value;
            }

[Copilot]

Pull request overview

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

Comments have been addressed. Dismissing because Ben is out of town and unavailable for a new review.