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

Fix #309; make images public #310

Closed
wants to merge 1 commit into from

Conversation

takahashim
Copy link
Collaborator

🎩 What? Why?

#309 では有効期限が長すぎてダメだったみたいなので、公開設定に変更する修正です。

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG upgrade notes, if required
  • If there's a new public field, add it to GraphQL API
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests

📷 Screenshots (optional)

@ayuki-joto
Copy link
Collaborator

こちら、おそらくs3の設定いじらないとなので一旦様子見です...

@takahashim
Copy link
Collaborator Author

DecidimでのS3の設定方法については以下のPlatoniqのAWS設定がありますが、これだと全公開してるっぽいですよね…。
https://github.com/Platoniq/decidim-install/blob/main/decidim-aws.md

@ayuki-joto
Copy link
Collaborator

流石にall actionは、許可したくないですね...
一応policyいじってみて、access自体は解決できそうなのですが、現在のCarrierWaveで発行されるURLが、
fog_publicをfalseにすると署名付きのものになるので、upload自体も署名なしでできるようにしないといけなそう...
それかuploadは今のままで、保存時にurlをparseして、署名部分省くなどの処理行えばとりあえず解決はできそう...

それか、cloudFrontに置き換えると期限を無制限にできそうなのでここを検討するのはアリかもしれません
https://stackoverflow.com/questions/9956712/use-cdn-with-carrierwave-fog-in-s3-cloudfront-with-rails-3-1
こちらも自前でparser挟まないといけなそうですが...

@ayuki-joto
Copy link
Collaborator

こちら機能的には修正できたのでcloseしますね

@ayuki-joto ayuki-joto closed this Dec 28, 2021
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

Successfully merging this pull request may close these issues.

2 participants