fix(stacks.api): correct filter on value, not path

Signed-off-by: Braden Mars <[email protected]>
CrisisCleanup · Aug 8, 2023 · 5a6dfb8 · 5a6dfb8
1 parent 4853445
commit 5a6dfb8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/packages/stacks/api/src/addons/crisiscleanup.ts b/packages/stacks/api/src/addons/crisiscleanup.ts
@@ -86,11 +86,11 @@ export class CrisisCleanupAddOn implements blueprints.ClusterAddOn {
 			{ path: 'host', objectAlias: 'POSTGRES_HOST' },
 		]
 
-		const externalDbKeys = dbSecretPaths.map(({ path }) =>
-			['postgres', path].join('.'),
+		const externalDbAliases = dbSecretPaths.map(
+			({ objectAlias }) => objectAlias,
 		)
 		const secretPaths: blueprints.JmesPathObject[] = Object.entries(secretKeys)
-			.filter(([key, _]) => !externalDbKeys.includes(key))
+			.filter(([_, value]) => !externalDbAliases.includes(value))
 			.map(([key, value]) => ({
 				path: ['api', 'secrets', key].join('.'),
 				objectAlias: value,

diff --git a/packages/stacks/api/test/__snapshots__/main.spec.ts.snap b/packages/stacks/api/test/__snapshots__/main.spec.ts.snap