Skip to content

Commit

Permalink
Merge branch 'main' into chore/fixing-merge-conflicts
Browse files Browse the repository at this point in the history
  • Loading branch information
TheOrangePuff committed Oct 17, 2023
2 parents cec245a + 9cb425a commit 0e8f2e8
Showing 1 changed file with 10 additions and 7 deletions.
17 changes: 10 additions & 7 deletions packages/static-hosting/lib/static-hosting.ts
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,9 @@ export interface ResponseHeaderMappings {
type Writeable<T> = { -readonly [P in keyof T]: T[P] };

export class StaticHosting extends Construct {
public readonly distribution: IDistribution;
public readonly bucket: IBucket;

private staticFiles = [
"js",
"css",
Expand Down Expand Up @@ -172,7 +175,7 @@ export class StaticHosting extends Construct {
});
}

const bucket = new Bucket(this, "ContentBucket", {
this.bucket = new Bucket(this, "ContentBucket", {
bucketName: siteName,
encryption: BucketEncryption.S3_MANAGED,
blockPublicAccess: BlockPublicAccess.BLOCK_ALL,
Expand All @@ -185,7 +188,7 @@ export class StaticHosting extends Construct {
comment: "Allow CloudFront to access S3",
});

bucket.grantRead(oai);
this.bucket.grantRead(oai);

new CfnOutput(this, "Bucket", {
description: "BucketName",
Expand All @@ -212,7 +215,7 @@ export class StaticHosting extends Construct {
: undefined;

if (publisherGroup) {
bucket.grantReadWrite(publisherGroup);
this.bucket.grantReadWrite(publisherGroup);

new CfnOutput(this, "PublisherGroupName", {
description: "PublisherGroup",
Expand Down Expand Up @@ -394,7 +397,7 @@ export class StaticHosting extends Construct {
"cloudfront:ListInvalidations",
],
resources: [
`arn:aws:cloudfront::*:distribution/${distribution.distributionId}`,
`arn:aws:cloudfront::*:distribution/${this.distribution.distributionId}`,
],
});

Expand All @@ -409,12 +412,12 @@ export class StaticHosting extends Construct {
}
new CfnOutput(this, "DistributionId", {
description: "DistributionId",
value: distribution.distributionId,
value: this.distribution.distributionId,
exportName: `${exportPrefix}DistributionID`,
});
new CfnOutput(this, "DistributionDomainName", {
description: "DistributionDomainName",
value: distribution.distributionDomainName,
value: this.distribution.distributionDomainName,
exportName: `${exportPrefix}DistributionName`,
});

Expand All @@ -425,7 +428,7 @@ export class StaticHosting extends Construct {

new ARecord(this, "SiteAliasRecord", {
recordName: siteName,
target: RecordTarget.fromAlias(new CloudFrontTarget(distribution)),
target: RecordTarget.fromAlias(new CloudFrontTarget(this.distribution)),
zone: zone,
});
}
Expand Down

0 comments on commit 0e8f2e8

Please sign in to comment.