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

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.

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

No branches or pull requests

1 participant