Revert "Revert change to make RequestRefresh run in the background (#…

…3083)" This reverts commit 3f2fbf1.
AzureAD · Jan 17, 2025 · 26a5bb0 · 26a5bb0
1 parent 6146f1f
commit 26a5bb0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs b/src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs
@@ -316,7 +316,7 @@ public override void RequestRefresh()
                 _isFirstRefreshRequest = false;
                 if (Interlocked.CompareExchange(ref _configurationRetrieverState, ConfigurationRetrieverRunning, ConfigurationRetrieverIdle) == ConfigurationRetrieverIdle)
                 {
-                    UpdateCurrentConfiguration();
+                    _ = Task.Run(UpdateCurrentConfiguration, CancellationToken.None);
                     _lastRequestRefresh = now;
                 }
             }

diff --git a/test/Microsoft.IdentityModel.Protocols.OpenIdConnect.Tests/ConfigurationManagerTests.cs b/test/Microsoft.IdentityModel.Protocols.OpenIdConnect.Tests/ConfigurationManagerTests.cs
@@ -230,7 +230,7 @@ public async Task VerifyInterlockGuardForRequestRefresh()
             // Interlocked guard will block.
             // Configuration should be AADCommonV1Config
             signalEvent.Reset();
-            _ = Task.Run(() => configurationManager.RequestRefresh());
+            configurationManager.RequestRefresh();
 
             // InMemoryDocumentRetrieverWithEvents will signal when it is OK to change the MetadataAddress
             // otherwise, it may be the case that the MetadataAddress is changed before the previous Task has finished.
@@ -239,7 +239,7 @@ public async Task VerifyInterlockGuardForRequestRefresh()
             // AADCommonV1Json would have been passed to the the previous retriever, which is blocked on an event.
             configurationManager.MetadataAddress = "AADCommonV2Json";
             TestUtilities.SetField(configurationManager, "_lastRequestRefresh", DateTimeOffset.MinValue);
-            _ = Task.Run(() => configurationManager.RequestRefresh());
+            configurationManager.RequestRefresh();
 
             // Set the event to release the lock and let the previous retriever finish.
             waitEvent.Set();
@@ -658,13 +658,14 @@ public async Task GetConfigurationAsync()
             var configuration = await configManager.GetConfigurationAsync(CancellationToken.None);
 
             TestUtilities.SetField(configManager, "_lastRequestRefresh", DateTimeOffset.UtcNow - TimeSpan.FromHours(1));
-            configManager.MetadataAddress = "http://127.0.0.1";
             configManager.RequestRefresh();
+            configManager.MetadataAddress = "http://127.0.0.1";
             var configuration2 = await configManager.GetConfigurationAsync(CancellationToken.None);
             IdentityComparer.AreEqual(configuration, configuration2, context);
             if (!object.ReferenceEquals(configuration, configuration2))
                 context.Diffs.Add("!object.ReferenceEquals(configuration, configuration2)");
 
+
             // get configuration from http address, should throw
             // get configuration with unsuccessful HTTP response status code
             TestUtilities.AssertFailIfErrors(context);