Remove CookieComplianceEnabled flag (#4588)

NuGet · Aug 23, 2017 · 4771a58 · 4771a58
1 parent f57f82f
commit 4771a58
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 16 deletions.
diff --git a/src/NuGetGallery/App_Start/DefaultDependenciesModule.cs b/src/NuGetGallery/App_Start/DefaultDependenciesModule.cs
@@ -537,11 +537,7 @@ private static void RegisterAuditingServices(ContainerBuilder builder, IAuditing
 
         private static void RegisterCookieComplianceService(ContainerBuilder builder, ConfigurationService configuration, DiagnosticsService diagnostics)
         {
-            CookieComplianceServiceBase service = null;
-            if (configuration.Current.CookieComplianceEnabled)
-            {
-                service = GetAddInServices<ICookieComplianceService>(builder).FirstOrDefault() as CookieComplianceServiceBase;
-            }
+            var service = GetAddInServices<ICookieComplianceService>(builder).FirstOrDefault() as CookieComplianceServiceBase;
 
             if (service == null)
             {

diff --git a/src/NuGetGallery/Configuration/AppConfiguration.cs b/src/NuGetGallery/Configuration/AppConfiguration.cs
@@ -19,9 +19,6 @@ public class AppConfiguration : IAppConfiguration
         [DefaultValue("")]
         public string RedesignBanner { get; set; }
 
-        [DefaultValue(false)]
-        public bool CookieComplianceEnabled { get; set; }
-
         /// <summary>
         /// Gets a setting indicating if SSL is required for all operations once logged in.
         /// </summary>

diff --git a/src/NuGetGallery/Configuration/IAppConfiguration.cs b/src/NuGetGallery/Configuration/IAppConfiguration.cs
@@ -27,11 +27,6 @@ public interface IAppConfiguration
         /// Gets the warning banner text
         /// </summary>
         string RedesignBanner { get; set; }
-
-        /// <summary>
-        /// Gets a setting if cookie compliance shim should be enabled.
-        /// </summary>
-        bool CookieComplianceEnabled { get; set; }
 
         /// <summary>
         /// Gets a setting indicating if SSL is required for all operations once logged in.

diff --git a/src/NuGetGallery/Web.config b/src/NuGetGallery/Web.config
@@ -50,9 +50,6 @@
     <add key="Gallery.WarningBanner" value="" />
     <!-- redesign banner on the top - leave blank to remove the banner from the site. -->
     <add key="Gallery.RedesignBanner" value="" />
-
-    <!-- EU cookie banner integration settings -->
-    <add key="Gallery.CookieComplianceEnabled" value="false" />
 
     <add key="Gallery.FacebookAppId" value="" />
     <!-- Set this if you have a Application Insights instrumentation key to use for your gallery deployment -->