Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bucket does not allow ACLs #42

Open
sarink opened this issue Jul 6, 2023 · 4 comments
Open

Bucket does not allow ACLs #42

sarink opened this issue Jul 6, 2023 · 4 comments
Assignees
Labels
kind/bug Some behavior is incorrect or out of spec
Milestone

Comments

@sarink
Copy link

sarink commented Jul 6, 2023

What happened?

AWS no longer allows ACLs for new buckets. However, the acl key is required on synced.S3BucketFolder, making it unusable

Expected Behavior

Expected creating an s3 bucket folder to work

Steps to reproduce

    new synced.S3BucketFolder(`${NAME}-synced-folder`, {
      path: '../../web/dist',
      bucketName: 'website',
      acl: aws.s3.PublicReadAcl,
    });

Output of pulumi about

CLI          
Version      3.74.0
Go Version   go1.20.5
Go Compiler  gc

Plugins
NAME    VERSION
nodejs  unknown

Host     
OS       darwin
Version  13.4
Arch     arm64

This project is written in nodejs: executable='/Users/kabirsarin/.nvm/versions/node/v18.16.0/bin/node' version='v18.16.0'

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@sarink sarink added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Jul 6, 2023
@sarink sarink closed this as completed Jul 6, 2023
@sarink sarink reopened this Jul 9, 2023
@sarink
Copy link
Author

sarink commented Jul 9, 2023

How can you enable ACLs for an S3 bucket with pulumi? Looking for this option

image

@gbegher
Copy link

gbegher commented Jul 17, 2023

@sarink

Here's what worked for me:

new s3.BucketOwnershipControls("bucket-ownership-controls", {
    bucket: THE_BUCKET,
    // For options check https://docs.aws.amazon.com/AmazonS3/latest/userguide/about-object-ownership.html
    rule: { objectOwnership: "ObjectWriter" },
})

new s3.BucketPublicAccessBlock("bucket-public-access-block", {
    bucket: THE_BUCKET,
    // allows the object writer to set public ACLs
    blockPublicAcls: false,
})

(found on https://blog.rishabkumar.com/deploying-a-static-website-to-aws-with-pulumi)

@scottslowe scottslowe removed the needs-triage Needs attention from the triage team label Jul 31, 2023
@cnunciato cnunciato self-assigned this Jul 1, 2024
@cnunciato cnunciato added this to the 0.107 milestone Jul 1, 2024
@github-project-automation github-project-automation bot moved this to 🤔 Triage in Docs 📚 Jul 1, 2024
@cnunciato cnunciato moved this from 🤔 Triage to 🔧 In Progress in Docs 📚 Jul 1, 2024
@cnunciato cnunciato moved this from 🔧 In Progress to 🎬 Ready in Docs 📚 Jul 1, 2024
@jonasgroendahl
Copy link

same issue here

@interurban interurban modified the milestones: 0.107, 0.109, 0.108 Jul 24, 2024
@interurban interurban modified the milestones: 0.108, 0.109 Aug 21, 2024
@interurban interurban modified the milestones: 0.109, 0.110 Sep 13, 2024
@interurban interurban modified the milestones: 0.110, 0.111 Oct 1, 2024
@interurban interurban modified the milestones: 0.111, 0.112 Oct 30, 2024
@thoward thoward modified the milestones: 0.112, 0.113 Nov 13, 2024
@thoward thoward assigned interurban and unassigned cnunciato Nov 13, 2024
@thoward
Copy link

thoward commented Nov 13, 2024

Need to figure out ownership of synced folders longterm.

@interurban interurban modified the milestones: 0.113, 0.114 Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec
Projects
Status: 🎬 Ready
Development

No branches or pull requests

7 participants