セキュアアップロードが無効な場合でもUpdatePostUploadsSecureStatusを実行する理由

saulshanabrook · 2023 年 2 月 18 日午後 4:45

Minio の S3 バックエンドのサポートありがとうございます。順調に動作しています。

唯一の問題は、Minio が ACL をサポートしていないため、UpdatePostUploadsSecureStatus ジョブが失敗することです。

github.com/minio/minio

Support Object ACLs

opened 02:26PM - 06 Sep 19 UTC

closed 03:58AM - 07 Sep 19 UTC

rfornea

community out of scope triage

I've read through a few github issues and the minio docs, and it seems you don't… support setting object ACLs (nor intend to). You claim to be an S3 compatible server, but if you aren't supporting this feature, that's not really true IMO. Regardless of whether or not AWS recommends using ACLs, they nevertheless still support them and it is a good enough solution for plenty of people's usage. Not supporting object ACLs prevents Minio from being a drop-in replacement for S3. Someone who wanted to switch to Minio would have to start adding prefixes to everything which could break their existing naming scheme for objects. This is a problem if they wanted to be able to support objects stored in S3 and in Minio, and if they already had pre-existing data in S3 which used Object ACLs. This is the situation I personally am in. I really think you should reconsider this.

このジョブの実行を無効にする方法はありますか？

Falco · 2023 年 2 月 18 日午後 4:51

SiteSetting.secure_uploads を有効にしましたか？

saulshanabrook · 2023 年 2 月 19 日午後 1:08

いいえ、していません。

編集：メールに埋め込まれた画像もいくつかインポートしました。メールでのセキュアアップロードのドキュメントを読んでいますが、これはメールからのすべてのアップロードが「セキュア」になるということでしょうか？

Falco · 2023 年 2 月 19 日午後 1:35

設定が無効になっている場合でも、すべての投稿のすべてのアップロードに対してこのジョブをスケジュールするのはなぜですか？ @martin

martin · 2023 年 2 月 19 日午後 11:38

良い質問です。これは2か所から呼び出されています。PostCreator:

github.com/discourse/discourse

lib/post_creator.rb

3c57db5c6


      
          def update_uploads_secure_status
            @post.update_uploads_secure_status(source: "post creator") if SiteSetting.secure_uploads?
          end

そしてPostRevisor:

github.com/discourse/discourse

lib/post_revisor.rb

3c57db5c6


      
          # This must be done before post_process_post, because that uses
          # post upload security status to cook URLs.
          @post.update_uploads_secure_status(source: "post revisor")

しかし、後者ではSiteSetting.secure_uploads?のチェックを怠っていたことがお分かりいただけるかと思います。これを修正するためにPRを作成し、チェックをpost.update_uploads_secure_statusに移動させる予定です。

編集：PRはこちらです。本日中にマージされることを願っています。FIX: Do not enqueue UpdatePostUploadsSecureStatus unnecessarily by martin-brennan · Pull Request #20366 · discourse/discourse · GitHub

トピック		返信	表示
Minio: A header you provided implies S3 functionality that is not implemented Support	20	9979	2021 年 4 月 5 日
Can not edit topics with picture with S3 backend Bug	11	1113	2021 年 7 月 12 日
Secure Uploads Announcements secure-uploads	46	16691	2025 年 7 月 24 日
Upload objects to private S3 is not working Support s3	5	2536	2022 年 6 月 14 日
Image upload error: The bucket does not allow ACL's Support s3	5	1713	2022 年 6 月 10 日

セキュアアップロードが無効な場合でもUpdatePostUploadsSecureStatusを実行する理由

関連トピック