Skip to content

Commit

Permalink
fix: ensure all SQS actions use SSL (#141)
Browse files Browse the repository at this point in the history
* fix: ensure all SQS actions use SSL

* docs: update increase security readme example

* 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
bestickley and github-actions authored Oct 2, 2023
1 parent 60da6e2 commit c41e1d1
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 19 deletions.
9 changes: 0 additions & 9 deletions API.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 0 additions & 9 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,15 +57,6 @@ export class UiStack {
new Nextjs(this, "NextSite", {
nextjsPath: "...",
defaults: {
assetDeployment: {
bucket: new Bucket(this, "NextjsAssetDeploymentBucket", {
autoDeleteObjects: true,
removalPolicy: RemovalPolicy.DESTROY,
encryption: BucketEncryption.S3_MANAGED,
enforceSSL: true,
blockPublicAccess: BlockPublicAccess.BLOCK_ALL,
}),
},
distribution: {
functionUrlAuthType: FunctionUrlAuthType.AWS_IAM,
cdk: {
Expand Down
2 changes: 1 addition & 1 deletion src/NextjsRevalidation.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export class NextjsRevalidation extends Construct {
queue.addToResourcePolicy(
new PolicyStatement({
sid: 'DenyUnsecureTransport',
actions: ['sqs:SendMessage', 'sqs:ReceiveMessage'],
actions: ['sqs:*'],
effect: Effect.DENY,
principals: [new AnyPrincipal()],
resources: [queue.queueArn],
Expand Down

0 comments on commit c41e1d1

Please sign in to comment.