flownative / flow-aws-s3

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

Fix acl target settings #48

Closed batabana closed 3 years ago

batabana commented 3 years ago

In the last commits concerning acl (either disabling it completely or setting it via config) there was a little mixup i guess - an old check $this->accessPolicyEnabled !== false was left behind at one place even thought the correspoding config option was supposed to be removed. Also the acl setting in the config got overwritten by the default setting. Additionally (for our use case) setting the acl to empty string was not enough, we explictly needed to not set it at all.