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.

Sign up for GitHub

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

Jump to bottom

[Storage] mkdir() error writing on a Uniform bucket-level access #2540

Closed
javiermarinros opened this issue Jan 7, 2020 · 4 comments · Fixed by #2541
Closed

[Storage] mkdir() error writing on a Uniform bucket-level access #2540

javiermarinros opened this issue Jan 7, 2020 · 4 comments · Fixed by #2541
Assignees
@dwsupplee
Labels
api: storage Issues related to the Cloud Storage API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@javiermarinros
Copy link

javiermarinros commented Jan 7, 2020 •
edited
Loading

Using mkdir() through the stream wrapper on a bucket with uniform level access, this ServiceException is thrown:

Cannot insert legacy ACL for an object when uniform bucket-level access is enabled. Read more at https://cloud.google.com/storage/docs/uniform-bucket-level-access.
@jdpedrie jdpedrie added api: storage Issues related to the Cloud Storage API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Jan 7, 2020
@jdpedrie jdpedrie mentioned this issue Jan 7, 2020
Merged
@jdpedrie
Copy link
Contributor

Sorry for not responding here @javiermarinros. In case you missed it, a fix for this is available in google/cloud-storage v1.17.0.

@javiermarinros
Copy link
Author

It's okay, I checked the same day you pushed the fix and everything works fine now. Thank you!

@trovster
Copy link

I have just deployed a new instance and I am getting the same error. I am using a Laravel wrapper, however, I have checked the dependencies and it is using v1.18.0 of the cloud storage.

@neoacevedo
Copy link

neoacevedo commented Jul 25, 2020 •
edited
Loading

I'm using the latest version, and mkdir returns false.

Debugging, I get the same ServiceException.
Ok forget it. I have to re-add the admin storage policy to the account, but the "bug" in this case comes from the Admin Console.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dwsupplee dwsupplee

Labels
api: storage Issues related to the Cloud Storage API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: allow mkdir() via stream wrapper when UBL is enabled
5 participants
@trovster @jdpedrie @neoacevedo @javiermarinros @dwsupplee