Allocation cleanup

.github/pull_request_template.md

@@ -0,0 +1,7 @@
+## Why this PR?
+
+-motivation for making this change-
+
+## Visible Changes
+
+-changes that are visible to developers using this library-

examples/ConsoleApp/ConsoleApp.csproj

@@ -7,9 +7,9 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
-    <PackageReference Include="System.Text.Json" Version="8.0.4" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.1" />
+    <PackageReference Include="System.Text.Json" Version="8.0.5" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.1" />
   </ItemGroup>
 
   <ItemGroup>

examples/ConsoleApp/appsettings.json

@@ -17,4 +17,4 @@
       }
     ]
   }
-}
+}

examples/TargetingConsoleApp/TargetingConsoleApp.csproj

@@ -7,9 +7,9 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.0" />
-    <PackageReference Include="System.Text.Json" Version="8.0.4" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />
+    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="8.0.1" />
+    <PackageReference Include="System.Text.Json" Version="8.0.5" />
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.1" />
   </ItemGroup>
 
   <ItemGroup>

src/Microsoft.FeatureManagement.AspNetCore/FeatureGateAttribute.cs

@@ -95,7 +95,7 @@ public FeatureGateAttribute(RequirementType requirementType, params object[] fea
         public RequirementType RequirementType { get; }
 
         /// <summary>
-        /// Performs controller action pre-procesing to ensure that any or all of the specified features are enabled.
+        /// Performs controller action pre-processing to ensure that any or all of the specified features are enabled.
         /// </summary>
         /// <param name="context">The context of the MVC action.</param>
         /// <param name="next">The action delegate.</param>

src/Microsoft.FeatureManagement/ConfigurationFeatureDefinitionProvider.cs

@@ -24,7 +24,7 @@ public sealed class ConfigurationFeatureDefinitionProvider : IFeatureDefinitionP
         // provider to be marked for caching as well.
 
         private readonly IConfiguration _configuration;
-        private readonly ConcurrentDictionary<string, FeatureDefinition> _definitions;
+        private readonly ConcurrentDictionary<string, Task<FeatureDefinition>> _definitions;
         private IDisposable _changeSubscription;
         private int _stale = 0;
 
@@ -37,7 +37,7 @@ public sealed class ConfigurationFeatureDefinitionProvider : IFeatureDefinitionP
         public ConfigurationFeatureDefinitionProvider(IConfiguration configuration)
         {
             _configuration = configuration ?? throw new ArgumentNullException(nameof(configuration));
-            _definitions = new ConcurrentDictionary<string, FeatureDefinition>();
+            _definitions = new ConcurrentDictionary<string, Task<FeatureDefinition>>();
 
             _changeSubscription = ChangeToken.OnChange(
                 () => _configuration.GetReloadToken(),
@@ -86,10 +86,13 @@ public Task<FeatureDefinition> GetFeatureDefinitionAsync(string featureName)
                 _definitions.Clear();
             }
 
-            return Task.FromResult(
-                _definitions.GetOrAdd(
-                    featureName,
-                    (_) => GetMicrosoftSchemaFeatureDefinition(featureName) ?? GetDotnetSchemaFeatureDefinition(featureName)));
+            if (!_definitions.ContainsKey(featureName))
+            {
+                _definitions[featureName] =
+                    Task.FromResult(GetMicrosoftSchemaFeatureDefinition(featureName) ?? GetDotnetSchemaFeatureDefinition(featureName));
+            }
+
+            return _definitions[featureName];
         }
 
         /// <summary>
@@ -98,7 +101,7 @@ public Task<FeatureDefinition> GetFeatureDefinitionAsync(string featureName)
         /// <returns>An enumerator which provides asynchronous iteration over feature definitions.</returns>
         //
         // The async key word is necessary for creating IAsyncEnumerable.
-        // The need to disable this warning occurs when implementaing async stream synchronously. 
+        // The need to disable this warning occurs when implementing async stream synchronously. 
 #pragma warning disable CS1998 // Async method lacks 'await' operators and will run synchronously
         public async IAsyncEnumerable<FeatureDefinition> GetAllFeatureDefinitionsAsync()
 #pragma warning restore CS1998
@@ -121,7 +124,17 @@ public async IAsyncEnumerable<FeatureDefinition> GetAllFeatureDefinitionsAsync()
 
                 //
                 // Underlying IConfigurationSection data is dynamic so latest feature definitions are returned
-                FeatureDefinition definition = _definitions.GetOrAdd(featureName, (_) => ParseMicrosoftSchemaFeatureDefinition(featureSection));
+                FeatureDefinition definition;
+
+                if (!_definitions.ContainsKey(featureName))
+                {
+                    definition = ParseMicrosoftSchemaFeatureDefinition(featureSection);
+                    _definitions[featureName] = Task.FromResult(definition);
+                }
+                else
+                {
+                    definition = _definitions[featureName].Result;
+                }
 
                 if (definition != null)
                 {
@@ -142,7 +155,17 @@ public async IAsyncEnumerable<FeatureDefinition> GetAllFeatureDefinitionsAsync()
 
                 //
                 // Underlying IConfigurationSection data is dynamic so latest feature definitions are returned
-                FeatureDefinition definition = _definitions.GetOrAdd(featureName, (_) => ParseDotnetSchemaFeatureDefinition(featureSection));
+                FeatureDefinition definition;
+
+                if (!_definitions.ContainsKey(featureName))
+                {
+                    definition = ParseDotnetSchemaFeatureDefinition(featureSection);
+                    _definitions[featureName] = Task.FromResult(definition);
+                }
+                else
+                {
+                    definition = _definitions[featureName].Result;
+                }
 
                 if (definition != null)
                 {