보안 업로드가 비활성화되어 있는데 왜 UpdatePostUploadsSecureStatus를 실행합니까?

saulshanabrook · 2월 18, 2023, 4:45오후

다양한 S3 백엔드를 지원하도록 작업해 주셔서 감사합니다! 저는 Minio를 사용하고 있으며, 잘 작동하고 있습니다…

단 하나의 작은 문제가 있는데, 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 · 2월 18, 2023, 4:51오후

SiteSetting.secure_uploads을 활성화하셨나요?

saulshanabrook · 2월 19, 2023, 1:08오후

아니요, 아직 아닙니다.

수정: 이미지가 포함된 일부 이메일을 가져왔습니다. 이메일의 안전한 업로드 문서를 읽고 있는데, 이는 이메일에서 온 모든 업로드가 “안전한” 상태가 된다는 뜻인가요?

Falco · 2월 19, 2023, 1:35오후

설정이 비활성화된 상태인데도, 왜 모든 게시물의 업로드마다 이 작업을 예약하는 거예요 @martin?

martin · 2월 19, 2023, 11:38오후

좋은 질문입니다 – 이 함수는 두 곳에서 호출됩니다. 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 - Pull Request #20366 - discourse/discourse - GitHub

글		댓글	조회수
Minio: A header you provided implies S3 functionality that is not implemented Product Support	19	10515	4월 5, 2021
Can not edit topics with picture with S3 backend Bug	10	1303	7월 9, 2021
Secure Uploads Announcements secure-uploads	43	18551	7월 24, 2025
Feature request: secure-uploads for S3 backends without per-object ACLs Feature s3 , secure-uploads	2	148	10월 18, 2026
Upload objects to private S3 is not working Product Support s3	5	2905	6월 14, 2022

보안 업로드가 비활성화되어 있는데 왜 UpdatePostUploadsSecureStatus를 실행합니까?

Related topics