Skip to content

Commit

Permalink
fix: don't add EncryptionKey for Sops if target is SecretsManager (#1044
Browse files Browse the repository at this point in the history
)

* fix: don't add EncryptionKey for Sops if target is SecretsManager

* chore: update snapshots

* chore: self mutation

Signed-off-by: github-actions <[email protected]>

---------

Signed-off-by: github-actions <[email protected]>
Co-authored-by: github-actions <[email protected]>
  • Loading branch information
markussiebert and github-actions committed Jul 15, 2024
1 parent 07e6fcd commit 4273367
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 3 additions & 1 deletion .tool-versions
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
golang 1.22.3
golang 1.22.3
yarn 1.22.19
nodejs 20.12.2
3 changes: 2 additions & 1 deletion src/SopsSync.ts
Original file line number Diff line number Diff line change
Expand Up @@ -365,7 +365,8 @@ export class SopsSync extends Construct {
Format: sopsFileFormat,
StringifiedValues: this.stringifiedValues,
ParameterName: props.parameterName,
EncryptionKey: props.encryptionKey?.keyId,
EncryptionKey:
props.secret !== undefined ? undefined : props.encryptionKey?.keyId,
},
});
this.versionId = cr.getAttString('VersionId');
Expand Down

0 comments on commit 4273367

Please sign in to comment.