From 54bd692c783426178beab34d08a7b09c48c73b4b Mon Sep 17 00:00:00 2001 From: Martin Costello Date: Tue, 7 Jul 2020 06:27:59 +0100 Subject: [PATCH] Use TryGetValue() in TokenExtensions (#23708) --- .../Authentication.Abstractions/src/TokenExtensions.cs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/Http/Authentication.Abstractions/src/TokenExtensions.cs b/src/Http/Authentication.Abstractions/src/TokenExtensions.cs index 0a04ae9aa3..e40bda6715 100644 --- a/src/Http/Authentication.Abstractions/src/TokenExtensions.cs +++ b/src/Http/Authentication.Abstractions/src/TokenExtensions.cs @@ -13,8 +13,8 @@ namespace Microsoft.AspNetCore.Authentication /// public static class AuthenticationTokenExtensions { - private static string TokenNamesKey = ".TokenNames"; - private static string TokenKeyPrefix = ".Token."; + private const string TokenNamesKey = ".TokenNames"; + private const string TokenKeyPrefix = ".Token."; /// /// Stores a set of authentication tokens, after removing any old tokens. @@ -76,9 +76,8 @@ namespace Microsoft.AspNetCore.Authentication } var tokenKey = TokenKeyPrefix + tokenName; - return properties.Items.ContainsKey(tokenKey) - ? properties.Items[tokenKey] - : null; + + return properties.Items.TryGetValue(tokenKey, out var value) ? value : null; } public static bool UpdateTokenValue(this AuthenticationProperties properties, string tokenName, string tokenValue)