Skip to content

Commit

Permalink
payloads bucket name cant be sanitized (#189)
Browse files Browse the repository at this point in the history
  • Loading branch information
jhsinger-klotho authored Feb 3, 2023
1 parent 6fafb81 commit e0aebbb
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions pkg/infra/pulumi_aws/deploylib.ts
Original file line number Diff line number Diff line change
Expand Up @@ -113,12 +113,8 @@ export class CloudCCLib {
if (this.createVPC) {
this.getVpcSgSubnets()
}
const resolvedBucketName = this.account.accountId.apply(
(accountId) =>
sanitized(
AwsSanitizer.S3.bucket.nameValidation()
)`${accountId}${physicalPayloadsBucketName}`
)
// Right now we sanitize this bucket name in the compiler go code so we do not need to here (issue #188 & pro#51)
const resolvedBucketName = pulumi.interpolate`${this.account.accountId}${physicalPayloadsBucketName}`
this.createBuckets([resolvedBucketName], true)
this.addSharedPolicyStatement({
Effect: 'Allow',
Expand Down

0 comments on commit e0aebbb

Please sign in to comment.