Merge pull request #2 from NikiforovAll/feature/development-only-by-default

Make DevelopmentOnly flag default option

Author: NikiforovAll

Source/ConfigurationDebugViewEndpoint/ConfigurationDebugViewEndpoint.csproj

@@ -12,7 +12,7 @@
   <ItemGroup>
     <FrameworkReference Include="Microsoft.AspNetCore.App" />
   </ItemGroup>
-  <ItemGroup Label="Pacakge References">
+  <ItemGroup Label="Package References">
     <PackageReference Include="Microsoft.Extensions.Configuration.Abstractions" Version="5.0.0" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="5.0.0" />
   </ItemGroup>

Source/ConfigurationDebugViewEndpoint/ConfigurationDebugViewOptions.cs

@@ -9,6 +9,6 @@ public class ConfigurationDebugViewOptions
         /// Gets or sets a value that controls whether to use <see cref="ConfigurationDebugViewMiddleware"/>
         /// exclusively on Development environment.
         /// </summary>
-        public bool AllowDevelopmentOnly { get; set; }
+        public bool AllowDevelopmentOnly { get; set; } = true;
     }
 }

Tests/ConfigurationDebugViewEndpoint.Test/EndpointRouteBuilderExtensionsTests.cs

@@ -81,5 +81,33 @@ public async Task WithAllowDevelopmentOnlyOptions_Development_ConfigurationRetur
             Assert.Equal(expectedStatusCode, (int)response.StatusCode);
         }
 
+        [Theory]
+        [InlineData("Development", 200)]
+        [InlineData("Production", 404)]
+        public async Task DefaultOptions_Development_ConfigurationReturned(
+            string environment,
+            int expectedStatusCode)
+        {
+            var pattern = "/config";
+            using var host = new HostBuilder().ConfigureWebHost(builder =>
+            {
+                builder.ConfigureServices(services => services.AddRouting());
+                builder.Configure(app =>
+                {
+                    app.UseRouting();
+                    app.UseEndpoints(endpoints =>
+                        endpoints.MapConfigurationDebugView(pattern));
+                });
+                builder.UseTestServer()
+                    .UseEnvironment(environment);
+            })
+                .Build();
+
+            await host.StartAsync();
+            using var server = host.GetTestServer();
+            var response = await server.CreateClient().GetAsync($"http://example.com{pattern}");
+            Assert.Equal(expectedStatusCode, (int)response.StatusCode);
+        }
+
     }
 }