fix: secrets manager logging (#360)

aws · Jan 2, 2025 · 4039c5a · 4039c5a
1 parent dda3977
commit 4039c5a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/common/lib/authentication/aws_secrets_manager_plugin.ts b/common/lib/authentication/aws_secrets_manager_plugin.ts
@@ -53,7 +53,7 @@ export class AwsSecretsManagerPlugin extends AbstractConnectionPlugin {
     const config: SecretsManagerClientConfig = {};
 
     if (!secretId) {
-      throw new AwsWrapperError(Messages.get("AwsSecretsManagerConnectionPlugin.missingRequiredConfigParameter"), WrapperProperties.SECRET_ID.name);
+      throw new AwsWrapperError(Messages.get("AwsSecretsManagerConnectionPlugin.missingRequiredConfigParameter", WrapperProperties.SECRET_ID.name));
     }
 
     if (!region) {

diff --git a/tests/unit/aws_secrets_manager_plugin.test.ts b/tests/unit/aws_secrets_manager_plugin.test.ts
@@ -120,7 +120,7 @@ describe("testSecretsManager", () => {
   it("missing required parameters", () => {
     expect(async () => {
       await new AwsSecretsManagerPlugin(mockPluginService, new Map()).connect(TEST_HOSTINFO, TEST_PROPS, true, mockConnectFunction);
-    }).rejects.toStrictEqual(new AwsWrapperError(Messages.get("AwsSecretsManagerConnectionPlugin.missingRequiredConfigParameter")));
+    }).rejects.toStrictEqual(new AwsWrapperError(Messages.get("AwsSecretsManagerConnectionPlugin.missingRequiredConfigParameter", "secretId")));
   });
 
   // The plugin will attempt to open a connection with a cached secret, but it will fail with a unhandled error.