search

flownative / flow-aws-s3

Amazon S3 adaptor for Neos and Flow
MIT License
18 stars 33 forks source link

FEATURE: support skipping unpublishing of resources #28

Closed skurfuerst closed 3 years ago

kitsunet commented 4 years ago

Probably makes sense for some applications.

batabana commented 3 years ago

Unfortunately I didn't see this PR before so I kind of added a new topic on top of it - sorry for the mess.

So to clear things up: I saw the review comment for the first feature (support skipping unpublishing of resources) just now and restructured the code accordingly. There was a new feature added before: Support disabling the standard ACL setting of the target (public-read) which is needed in some cases, e.g. when using cloudfront with conflicting restrictive policies.