Improve log when secret is not found in keyVault (#173)

This pull request introduces improvements to error handling and logging
when interacting with Azure Key Vault. The changes primarily focus on
providing more detailed error messages by including the path of the
secret that caused the error.

### Improvements to error handling:

*
[`src/Confix.Tool/src/Confix.Library/ThrowHelper.cs`](diffhunk://#diff-de2d44cf88807e7118a43465a75745e67e659f7268cad9c28496735aac87ad67L44-R48):
Modified the `SecretNotFound` method to accept an optional `path`
parameter and include it in the error message if provided.

### Updates to method signatures:

*
[`src/Confix.Tool/src/Confix.Library/Utilities/Azure/KeyVaultExtension.cs`](diffhunk://#diff-520f5459080dfd82d45c6e5bf94a9cc9a0ecda7e85b403aa953d50421aee0ebcL4-R17):
Updated the `HandleKeyVaultException` method to accept an optional
`path` parameter and pass it to the `SecretNotFound` method.

### Enhancements to Azure Key Vault provider:

*
[`src/Confix.Tool/src/Confix.Library/Variables/Providers/AzureKeyVault/AzureKeyVaultProvider.cs`](diffhunk://#diff-bcce2fb5047575d77b3d9a54ee0f9df0688c86e01fb6afd1665a8ae18b20f135L57-R57):
Updated the `ResolveAsync` and `SetAsync` methods to pass the `path`
parameter to the `HandleKeyVaultException` method.
[[1]](diffhunk://#diff-bcce2fb5047575d77b3d9a54ee0f9df0688c86e01fb6afd1665a8ae18b20f135L57-R57)
[[2]](diffhunk://#diff-bcce2fb5047575d77b3d9a54ee0f9df0688c86e01fb6afd1665a8ae18b20f135L75-R75)-
mention secret path in error message when available

Author: RohrerF

src/Confix.Tool/src/Confix.Library/ThrowHelper.cs

@@ -41,8 +41,11 @@ public static Exception VariablesNotFound(string[] names) =>
     public static Exception CouldNotParseJsonFile(FileInfo file)
         => throw new ExitException($"File {file.FullName} has invalid content.");
 
-    public static Exception SecretNotFound(Exception innerException) =>
-        new ExitException("Secret does not exist in this provider.", innerException)
+    public static Exception SecretNotFound(Exception innerException, string? path = null) =>
+        new ExitException(
+            path is null
+                ? "Secret does not exist in this provider."
+                : $"Secret {path.AsHighlighted()} does not exist in this provider.", innerException)
         {
             Help = $"try running {"confix variable list".AsHighlighted()} to list all available variables"
         };
@@ -53,7 +56,7 @@ public static Exception AccessToKeyVaultFailed(RequestFailedException innerExcep
         details.AppendLine($"Message: {innerException.Message}");
         details.AppendLine($"Error code: {innerException.ErrorCode}");
         details.AppendLine($"Status code: {innerException.Status}");
-        
+
         return new ExitException("Access to Key Vault failed", innerException)
         {
             Help = "check if you have the required permissions to access the Key Vault",
@@ -66,4 +69,4 @@ public static Exception AuthenticationFailedForVault(Exception innerException) =
         {
             Help = $"try running {"az login".AsHighlighted()} to authenticate with Azure"
         };
-}
+}

src/Confix.Tool/src/Confix.Library/Utilities/Azure/KeyVaultExtension.cs

@@ -1,21 +1,20 @@
 using Azure;
 using Azure.Identity;
 using Confix.Tool;
-using Confix.Tool.Commands.Logging;
 
 namespace Confix.Utilities.Azure;
 
 public static class KeyVaultExtension
 {
-    public static async Task<T> HandleKeyVaultException<T>(Func<Task<T>> action)
+    public static async Task<T> HandleKeyVaultException<T>(Func<Task<T>> action, string? path = null)
     {
         try
         {
             return await action();
         }
         catch (RequestFailedException ex) when (ex.ErrorCode == "SecretNotFound")
         {
-            throw ThrowHelper.SecretNotFound(ex);
+            throw ThrowHelper.SecretNotFound(ex, path);
         }
         catch (RequestFailedException ex)
         {
@@ -26,4 +25,4 @@ public static async Task<T> HandleKeyVaultException<T>(Func<Task<T>> action)
             throw ThrowHelper.AuthenticationFailedForVault(ex);
         }
     }
-}
+}

src/Confix.Tool/src/Confix.Library/Variables/Providers/AzureKeyVault/AzureKeyVaultProvider.cs

@@ -38,7 +38,7 @@ public Task<IReadOnlyList<string>> ListAsync(CancellationToken cancellationToken
         => KeyVaultExtension.HandleKeyVaultException<IReadOnlyList<string>>(async () =>
         {
             App.Log.ListSecrets(_client.VaultUri);
-            
+
             var secrets = new List<string>();
             await foreach (var secret in _client.GetPropertiesOfSecretsAsync(cancellationToken))
             {
@@ -54,7 +54,7 @@ public Task<JsonNode> ResolveAsync(string path, CancellationToken cancellationTo
             KeyVaultSecret result = await _client.GetSecretAsync(path.ToKeyVaultCompatiblePath(),
                 cancellationToken: cancellationToken);
             return JsonValue.Create(result.Value)!;
-        });
+        }, path);
 
     public Task<IReadOnlyDictionary<string, JsonNode>> ResolveManyAsync(
         IReadOnlyList<string> paths,
@@ -70,9 +70,9 @@ public Task<string> SetAsync(string path, JsonNode value, CancellationToken ct)
             }
 
             KeyVaultSecret result = await _client
-                .SetSecretAsync(path.ToKeyVaultCompatiblePath(), (string) value!, ct);
+                .SetSecretAsync(path.ToKeyVaultCompatiblePath(), (string)value!, ct);
             return result.Name.ToConfixPath();
-        });
+        }, path);
 
     public ValueTask DisposeAsync()
     {
@@ -93,4 +93,4 @@ public static void ListSecrets(this IConsoleLogger log, Uri vaultUri)
     {
         log.Information($"List all secrets from Azure Kev Vault '{vaultUri}'");
     }
-}
+}