Added comment why _negotiateAuth is reset to null when retrying.

arellegue · arellegue · commit a9c52edd9cdb · 2024-06-10T18:32:29.000Z
diff --git a/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SSPI/NegotiateSSPIContextProvider.cs b/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SSPI/NegotiateSSPIContextProvider.cs
@@ -25,7 +25,7 @@ internal override void GenerateSspiClientContext(ReadOnlyMemory<byte> received,
                 if (statusCode == NegotiateAuthenticationStatusCode.Completed || statusCode == NegotiateAuthenticationStatusCode.ContinueNeeded)
                     break;
                 else
-                    _negotiateAuth = null;
+                    _negotiateAuth = null; // Reset _negotiateAuth to be generated again for next SPN.
             }
 
             if (statusCode is not NegotiateAuthenticationStatusCode.Completed and not NegotiateAuthenticationStatusCode.ContinueNeeded)

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ internal override void GenerateSspiClientContext(ReadOnlyMemory<byte> received,`
`25`	`25`	`if (statusCode == NegotiateAuthenticationStatusCode.Completed \|\| statusCode == NegotiateAuthenticationStatusCode.ContinueNeeded)`
`26`	`26`	`break;`
`27`	`27`	`else`
`28`		`- _negotiateAuth = null;`
	`28`	`+ _negotiateAuth = null; // Reset _negotiateAuth to be generated again for next SPN.`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`if (statusCode is not NegotiateAuthenticationStatusCode.Completed and not NegotiateAuthenticationStatusCode.ContinueNeeded)`