chore: Disable ASP.NET Core 6+ browser injection by default. (#2052)

newrelic · Nov 9, 2023 · 887061a · 887061a
1 parent e571ac1
commit 887061a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Agent/NewRelic/Agent/Core/Configuration/DefaultConfiguration.cs b/src/Agent/NewRelic/Agent/Core/Configuration/DefaultConfiguration.cs
@@ -2103,7 +2103,7 @@ public bool EnableAspNetCore6PlusBrowserInjection
             {
                 return _enableAspNetCore6PlusBrowserInjection.HasValue
                     ? _enableAspNetCore6PlusBrowserInjection.Value
-                    : (_enableAspNetCore6PlusBrowserInjection = TryGetAppSettingAsBoolWithDefault("EnableAspNetCore6PlusBrowserInjection", true)).Value;
+                    : (_enableAspNetCore6PlusBrowserInjection = TryGetAppSettingAsBoolWithDefault("EnableAspNetCore6PlusBrowserInjection", false)).Value;
             }
         }
 

diff --git a/tests/Agent/UnitTests/Core.UnitTest/Configuration/DefaultConfigurationTests.cs b/tests/Agent/UnitTests/Core.UnitTest/Configuration/DefaultConfigurationTests.cs
@@ -3241,8 +3241,8 @@ public bool AsyncHttpClientSegmentsDoNotCountTowardsParentExclusiveTimeTests(str
             return defaultConfig.ForceSynchronousTimingCalculationHttpClient;
         }
 
-        [TestCase(null, ExpectedResult = true)]
-        [TestCase("not a bool", ExpectedResult = true)]
+        [TestCase(null, ExpectedResult = false)]
+        [TestCase("not a bool", ExpectedResult = false)]
         [TestCase("false", ExpectedResult = false)]
         [TestCase("true", ExpectedResult = true)]
         public bool AspNetCore6PlusBrowserInjectionTests(string localConfigValue)