remove duplicate keyid into var keysAttempted (#2899)

bdapoigny · jennyf19 · kllysng · web-flow · commit 82cd3fdf182e · 2025-01-05T23:51:01.000-08:00
Co-authored-by: jennyf19 &lt;jeferrie@microsoft.com&gt;
Co-authored-by: kellyyangsong &lt;69649063+kellyyangsong@users.noreply.github.com&gt;
Co-authored-by: Peter &lt;34331512+pmaytak@users.noreply.github.com&gt;
diff --git a/src/System.IdentityModel.Tokens.Jwt/JwtSecurityTokenHandler.cs b/src/System.IdentityModel.Tokens.Jwt/JwtSecurityTokenHandler.cs
@@ -1360,7 +1360,7 @@ private JwtSecurityToken ValidateSignature(string token, JwtSecurityToken jwtTok
 
                     if (key != null)
                     {
-                        (keysAttempted ??= new StringBuilder()).Append(key.ToString()).Append(" , KeyId: ").AppendLine(key.KeyId);
+                        (keysAttempted ??= new StringBuilder()).Append(key.ToString());
                         if (kidExists && !kidMatched && key.KeyId != null)
                             kidMatched = jwtToken.Header.Kid.Equals(key.KeyId, key is X509SecurityKey ? StringComparison.OrdinalIgnoreCase : StringComparison.Ordinal);
                     }

Original file line number	Diff line number	Diff line change
`@@ -1360,7 +1360,7 @@ private JwtSecurityToken ValidateSignature(string token, JwtSecurityToken jwtTok`
`1360`	`1360`
`1361`	`1361`	`if (key != null)`
`1362`	`1362`	`{`
`1363`		`- (keysAttempted ??= new StringBuilder()).Append(key.ToString()).Append(" , KeyId: ").AppendLine(key.KeyId);`
	`1363`	`+ (keysAttempted ??= new StringBuilder()).Append(key.ToString());`
`1364`	`1364`	`if (kidExists && !kidMatched && key.KeyId != null)`
`1365`	`1365`	`kidMatched = jwtToken.Header.Kid.Equals(key.KeyId, key is X509SecurityKey ? StringComparison.OrdinalIgnoreCase : StringComparison.Ordinal);`
`1366`	`1366`	`}`