Skip to content

Commit

Permalink
fix(lib): Change allowedAccountIds & forbiddenAccountIds in S3Backend…
Browse files Browse the repository at this point in the history
…Config to string[] to match Terraform

When providing these fields as a string rather than a list of strings, Terraform throws an error.
  • Loading branch information
Booligoosh authored and ansgarm committed Jul 15, 2024
1 parent b9f8c0b commit 021ddbb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packages/cdktf/lib/backends/s3-backend.ts
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ export interface S3BackendConfig {
* (Optional) List of allowed AWS account IDs to prevent potential destruction of a live environment.
* Conflicts with forbidden_account_ids.
*/
readonly allowedAccountIds?: string;
readonly allowedAccountIds?: string[];
/**
* (Optional) File containing custom root and intermediate certificates.
* Can also be set using the AWS_CA_BUNDLE environment variable.
Expand All @@ -91,7 +91,7 @@ export interface S3BackendConfig {
* (Optional) List of forbidden AWS account IDs to prevent potential destruction of a live environment.
* Conflicts with allowed_account_ids.
*/
readonly forbiddenAccountIds?: string;
readonly forbiddenAccountIds?: string[];
/**
* (Optional) URL of a proxy to use for HTTP requests when accessing the AWS API.
* Can also be set using the HTTP_PROXY or http_proxy environment variables.
Expand Down

0 comments on commit 021ddbb

Please sign in to comment.