Use the locking properly, this is not Python...

2019-02-07 21:05:58 +01:00 · 2019-02-07 21:05:58 +01:00 · d6835f8dd6
commit d6835f8dd6
parent 86940e96d5
1 changed files with 9 additions and 2 deletions
--- a/MediaBrowser.Providers/TV/TheTVDB/TvDbClientManager.cs
+++ b/MediaBrowser.Providers/TV/TheTVDB/TvDbClientManager.cs
@ -90,16 +90,23 @@ namespace MediaBrowser.Providers.TV
            {
                return cachedValue;
            }
-            using (_cacheWriteLock)
+
+            await _cacheWriteLock.WaitAsync().ConfigureAwait(false);
+            try
            {
                if (_cache.TryGetValue(key, out cachedValue))
                {
                    return cachedValue;
                }
+
                var result = await resultFactory.Invoke();
-                _cache.Set(key, result, DateTimeOffset.UtcNow.AddHours(1));
+                _cache.Set(key, result, TimeSpan.FromHours(1));
                return result;
            }
+            finally
+            {
+                _cacheWriteLock.Release();
+            }
        }
    }
 }