Skip to content

Fix SPN using instance name instead of resolved port for Protocol.None and Protocol.Admin #4180

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
paulmedynski wants to merge 3 commits into
base: main
Choose a base branch
from
+392 −6

Address Copilot review feedback: SPN safety, DNS isolation, and test …

d485228
Select commit
Loading
Failed to load commit list.
Draft

Fix SPN using instance name instead of resolved port for Protocol.None and Protocol.Admin #4180

Address Copilot review feedback: SPN safety, DNS isolation, and test …
d485228
Select commit
Loading
Failed to load commit list.
Azure Pipelines / PR-SqlClient-Package (MacOSLatest_Sql22 net8_0_AnyCPU_ManagedSNI_2) succeeded May 7, 2026 in 58m 57s

MacOSLatest_Sql22 net8_0_AnyCPU_ManagedSNI_2 succeeded

2 errors / 1 warnings

Annotations

Check failure on line 142 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / PR-SqlClient-Package (MacOSLatest_Sql22 net8_0_AnyCPU_ManagedSNI_2)

Build log #L142 

Error: The process '/Users/runner/hostedtoolcache/dotnet/dotnet' failed with exit code 1

Check failure on line 144 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / PR-SqlClient-Package (MacOSLatest_Sql22 net8_0_AnyCPU_ManagedSNI_2)

Build log #L144 

Dotnet command failed with non-zero exit code on the following projects : [ '/Users/runner/work/1/s/build.proj' ]
View more details on Azure Pipelines