Refactor Google Secrets integration to support options and improve configuration handling (#23)

This pull request introduces several significant changes to the
`GoogleSecrets` configuration functionality. The most important updates
include the addition of a new overload for `AddGoogleSecrets`, the
removal of redundant code, and enhancements to the handling of
`GoogleSecretsOptions`.

### Key Changes:

#### New Features:
* Added a new overload for `AddGoogleSecrets` to allow passing
`GoogleSecretsOptions`.

#### Code Simplification:
* Removed the redundant environment variable check for
`GoogleSecretsProject` in `ConfigurationBuilderExtensions`.
[[1]](diffhunk://#diff-3a42ad5a1b6e9fd57bf3513b6cdee8a15582556ed87bd3a456c3cff0da036c27L20-L29)
[[2]](diffhunk://#diff-4c1edd3a47fe0ee6052f244806defa0d9acf7a1e57660749e2a7de329850795eL20-R33)
* Simplified the `AddGoogleSecrets` method by removing the check for
`ProjectName` in `GoogleSecretsOptions`.

#### Enhancements:
* Set the `ProjectName` property in `GoogleSecretsOptions` to default to
the environment variable `GoogleSecretsProject` if available.
* Added a warning log and early return in `GoogleSecretsProvider` if
`ProjectName` is not set.

Author: neotrow

CHANGELOG.md

@@ -7,6 +7,10 @@ and adheres to a project-specific [Versioning](/README.md).
 
 ## [Unreleased]
 
+### Added
+
+- Added new overload for `AddGoogleSecrets` to allow passing `GoogleSecretsOptions`.
+
 ## [1.4.0] - 2025-03-18
 
 ### Added

GoogleSecrets/ConfigurationBuilderExtensions.cs

@@ -17,16 +17,6 @@ public static class ConfigurationBuilderExtensions
         /// <exception cref="System.ArgumentNullException">options</exception>
         public static IConfigurationBuilder AddGoogleSecrets(this IConfigurationBuilder configuration)
         {
-            // Configure app configuration to add Google Secrets if environment variable is set
-            var googleSecretProject = Environment.GetEnvironmentVariable(EnvironmentVariableNames.GoogleSecretsProject);
-            if (!string.IsNullOrWhiteSpace(googleSecretProject))
-            {
-                return AddGoogleSecrets(configuration, options =>
-                {
-                    options.ProjectName = googleSecretProject;
-                });
-            }
-
             return AddGoogleSecrets(configuration, _ => { });
         }
 
@@ -45,11 +35,6 @@ public static IConfigurationBuilder AddGoogleSecrets(this IConfigurationBuilder
             var googleSecretsOptions = new GoogleSecretsOptions();
             options(googleSecretsOptions);
 
-            if (string.IsNullOrWhiteSpace(googleSecretsOptions.ProjectName))
-            {
-                throw new ArgumentNullException(nameof(googleSecretsOptions.ProjectName));
-            }
-
             configuration.Add(new GoogleSecretsSource(googleSecretsOptions, configuration.Build()));
 
             return configuration;

GoogleSecrets/GoogleSecretsOptions.cs

@@ -41,7 +41,7 @@ public class GoogleSecretsOptions
         /// <value>
         /// The name of the project or the numeric project id
         /// </value>
-        public string ProjectName { get; set; }
+        public string ProjectName { get; set; } = Environment.GetEnvironmentVariable(EnvironmentVariableNames.GoogleSecretsProject) ?? string.Empty;
 
         /// <summary>
         /// Gets or sets the filter.

GoogleSecrets/GoogleSecretsProvider.cs

@@ -45,6 +45,12 @@ public override void Load()
         {
             try
             {
+                if (string.IsNullOrWhiteSpace(this.Source.ProjectName))
+                {
+                    this.logger.LogWarning("ProjectName is not set. Skipping Google Secrets Provider.");
+                    return;
+                }
+
                 // Create client
                 SecretManagerServiceClient secretManagerServiceClient = SecretManagerServiceClient.Create();
 

Neolution.Extensions.Configuration.GoogleSecrets.AspNetCore/WebApplicationBuilderExtensions.cs

@@ -17,15 +17,20 @@ public static void AddGoogleSecrets(this WebApplicationBuilder builder)
         {
             ArgumentNullException.ThrowIfNull(builder);
 
-            // Configure app configuration to add Google Secrets if environment variable is set
-            var googleSecretProject = Environment.GetEnvironmentVariable(EnvironmentVariableNames.GoogleSecretsProject);
-            if (!string.IsNullOrWhiteSpace(googleSecretProject))
-            {
-                builder.Configuration.AddGoogleSecrets(options =>
-                {
-                    options.ProjectName = googleSecretProject;
-                });
-            }
+            builder.Configuration.AddGoogleSecrets(_ => { });
+        }
+
+        /// <summary>
+        /// Adds the Google secrets to the <see cref="WebApplicationBuilder"/>.
+        /// </summary>
+        /// <param name="builder"></param>
+        /// <param name="googleSecretsOptions"></param>
+        public static void AddGoogleSecrets(this WebApplicationBuilder builder, Action<GoogleSecretsOptions> googleSecretsOptions)
+        {
+            ArgumentNullException.ThrowIfNull(builder);
+            ArgumentNullException.ThrowIfNull(googleSecretsOptions);
+
+            builder.Configuration.AddGoogleSecrets(googleSecretsOptions);
         }
     }
 }