PR Comments

aws · Jun 26, 2019 · 2c79776 · 2c79776
1 parent a41e9fb
commit 2c79776
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/AWS.Logger.AspNetCore/AWSLoggerBuilderExtensions.cs b/src/AWS.Logger.AspNetCore/AWSLoggerBuilderExtensions.cs
@@ -68,7 +68,9 @@ public static ILoggingBuilder AddAWSProvider(this ILoggingBuilder builder, Func<
         {
             var serviceDescriptor = builder.Services.FirstOrDefault(x => x.ServiceType == typeof(IConfiguration));
             if (serviceDescriptor == null)
+            {
                 return builder;
+            }
 
             var configuration = serviceDescriptor.ImplementationInstance as IConfiguration;
 
@@ -77,10 +79,15 @@ public static ILoggingBuilder AddAWSProvider(this ILoggingBuilder builder, Func<
             // and skip adding the provider. We don't want to prevent developers running their application
             // locally because they don't have access or want to use AWS for their local development.
             if (configuration == null)
+            {
                 return builder;
+            }
+
             var configSection = configuration.GetAWSLoggingConfigSection();
             if (configSection == null)
+            {
                 return builder;
+            }
 
             return AddAWSProvider(builder, configSection, formatter);
         }

diff --git a/src/AWS.Logger.AspNetCore/AWSLoggerConfigSection.cs b/src/AWS.Logger.AspNetCore/AWSLoggerConfigSection.cs
@@ -13,12 +13,12 @@ public static class ConfigurationSectionExtensions
     {
         // Default configuration block on the appsettings.json
         // Customer's information will be fetched from this block unless otherwise set.
-        const string DEFAULT_BLOCK = "Logging";
+        private const string DEFAULT_BLOCK = "Logging";
 
         // This library was originally written before logging standarized, or it at least we didn't realize it was standarized, on the "Logging" section in the config.
         // The library now uses "Logging" as the default section to look for config but to maintain backwards compatibility the package will fallback
         // AWS.Logging if a log group is not configured in the "Logging" config block".
-        const string LEGACY_DEFAULT_BLOCK = "AWS.Logging";
+        private const string LEGACY_DEFAULT_BLOCK = "AWS.Logging";
 
         /// <summary>
         /// Loads the AWS Logger Configuration from the ConfigSection