Warum UpdatePostUploadsSecureStatus ausführen, auch wenn sichere Uploads deaktiviert sind?

saulshanabrook · 18. Februar 2023 um 16:45

Vielen Dank für Ihre Arbeit an der Unterstützung verschiedener S3-Backends! Ich verwende Minio und es funktioniert hervorragend…

Das einzige kleine Problem, das ich habe, ist, dass der Job UpdatePostUploadsSecureStatus fehlschlägt, da Minio keine ACLs unterstützt:

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.

Gibt es eine Möglichkeit, diesen Job von der Ausführung auszuschließen?

Falco · 18. Februar 2023 um 16:51

Haben Sie SiteSetting.secure_uploads aktiviert?

saulshanabrook · 19. Februar 2023 um 13:08

Nein, das habe ich nicht.

EDIT: Ich habe einige E-Mails importiert, die Bilder eingebettet hatten. Ich lese gerade die Dokumentation Sichere Uploads in E-Mails. Sagt das, dass alle Uploads aus E-Mails „sicher“ sein werden?

Falco · 19. Februar 2023 um 13:35

Warum planen wir diesen Job für jeden Upload bei jedem Beitrag, auch wenn die Einstellung deaktiviert ist, @martin?

martin · 19. Februar 2023 um 23:38

Gute Frage – dies wird an zwei Stellen aufgerufen. 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

Und 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")

Wie Sie jedoch sehen können, habe ich es versäumt, die SiteSetting.secure_uploads?-Prüfung in letzterer durchzuführen … Ich werde einen PR erstellen, um dies zu beheben und die Prüfung einfach nach post.update_uploads_secure_status zu verschieben.

Bearbeiten: PR ist hier, hoffentlich wird er heute zusammengeführt FIX: Do not enqueue UpdatePostUploadsSecureStatus unnecessarily by martin-brennan · Pull Request #20366 · discourse/discourse · GitHub

Thema		Antworten	Aufrufe
Minio: A header you provided implies S3 functionality that is not implemented Support	20	9979	5. April 2021
Can not edit topics with picture with S3 backend Bug	11	1113	12. Juli 2021
Secure Uploads Announcements secure-uploads	46	16691	24. Juli 2025
Upload objects to private S3 is not working Support s3	5	2536	14. Juni 2022
Image upload error: The bucket does not allow ACL's Support s3	5	1713	10. Juni 2022

Warum UpdatePostUploadsSecureStatus ausführen, auch wenn sichere Uploads deaktiviert sind?

Verwandte Themen