From 8fa9d0a89cc3658d346fe293fc5d9b16cf99b69f Mon Sep 17 00:00:00 2001 From: Simon Cropp Date: Sun, 7 Apr 2024 19:24:11 +1000 Subject: [PATCH 1/2] remove redundant containskey --- src/Alba/Internal/LightweightCache.cs | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/Alba/Internal/LightweightCache.cs b/src/Alba/Internal/LightweightCache.cs index f072571a..2b1464ab 100644 --- a/src/Alba/Internal/LightweightCache.cs +++ b/src/Alba/Internal/LightweightCache.cs @@ -85,14 +85,7 @@ public TValue this[TKey key] } set { - if (_values.ContainsKey(key)) - { - _values[key] = value; - } - else - { - _values.Add(key, value); - } + _values[key] = value; } } From 1b43261509525703b49527d90b65324f803e3e8c Mon Sep 17 00:00:00 2001 From: Simon Cropp Date: Sun, 7 Apr 2024 19:26:55 +1000 Subject: [PATCH 2/2] . --- src/Alba/Internal/LightweightCache.cs | 10 +--------- src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs | 3 +-- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/src/Alba/Internal/LightweightCache.cs b/src/Alba/Internal/LightweightCache.cs index 2b1464ab..22ae4a36 100644 --- a/src/Alba/Internal/LightweightCache.cs +++ b/src/Alba/Internal/LightweightCache.cs @@ -121,15 +121,7 @@ public void Fill(TKey key, TValue value) public bool TryRetrieve(TKey key, [MaybeNullWhen(false)] out TValue value) { - value = default; - - if (_values.ContainsKey(key)) - { - value = _values[key]; - return true; - } - - return false; + return _values.TryGetValue(key, out value); } public void Each(Action action) diff --git a/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs b/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs index 4be98627..ec29904e 100644 --- a/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs +++ b/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs @@ -15,9 +15,8 @@ public ViewModel(AuthenticateResult result) { AuthenticateResult = result; - if (result.Properties.Items.ContainsKey("client_list")) + if (result.Properties.Items.TryGetValue("client_list", out var encoded)) { - var encoded = result.Properties.Items["client_list"]; var bytes = Base64Url.Decode(encoded); var value = Encoding.UTF8.GetString(bytes);