Skip to content
New issue

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

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

Already on GitHub? # to your account

fog-google: Cannot insert legacy ACL #121

Open
cwjenkins opened this issue Aug 7, 2023 · 0 comments
Open

fog-google: Cannot insert legacy ACL #121

cwjenkins opened this issue Aug 7, 2023 · 0 comments

Comments

@cwjenkins
Copy link

Describe the bug
Paperclip fog implementation sets fog_public to true if fog_public isn't specified (default value).
This causes fog-google to throw an exception when the corresponding bucket is using uniform access controls given predefined_acls (fine grain access control) is set (predefinedAcl=publicRead).
If fog_public were set to false then it also causes the same issue given predefinedAcl is set to private in fog-google.

Workaround: setting fog_public to a lambda that returns nil (seems hacky).

To Reproduce

  1. Create a GCS account.
  2. Create a bucket with uniform access control.
  3. Do not config fog_public
  4. Try saving an attachment.

Expected behavior
Not specifying fog_public leaves public nil.

Additional context
Issue on their end.
A PR providing a way to skip predefinedAcl if another gem sets it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant