dirien
14. Juni 2021 um 05:45
1
Hallo zusammen,
wir sind auf ein seltsames Problem gestoßen.
Unsere Konfiguration:
Discourse: 2.8.0.beta2
S3: Minio
CDN: Myra
Die S3-Konfiguration sieht wie folgt aus:
root@assets-prod-app01:~# sudo -u minio mc admin policy info minio readwrite-discourse
{
"Version": "2012-10-17",
"Statement": [
{
"Effect": "Allow",
"Action": [
"s3:ListBucketMultipartUploads",
"s3:GetBucketLocation",
"s3:ListBucket"
],
"Resource": [
"arn:aws:s3:::discourse"
]
},
{
"Effect": "Allow",
"Action": [
"s3:*"
],
"Resource": [
"arn:aws:s3:::discourse/*"
]
}
]
}
Doch beim Bearbeiten erhalten wir ständig folgende Fehlermeldung:
In den Log-Dateien taucht folgendes auf:
Aws::S3::Errors::NotImplemented (A header you provided implies functionality that is not implemented)
lib/file_store/s3_store.rb:279:in `update_ACL'
lib/file_store/s3_store.rb:209:in `update_upload_ACL'
...
Gibt es eine Einstellung, die wir übersehen haben, oder ist Minio nicht mit Discourse kompatibel?
Jeder Hinweis wäre hilfreich, da wir von unseren Nutzern zunehmend unter Druck geraten
Vielen Dank im Voraus
dirien
22. Juni 2021 um 04:24
2
Ist diese Einstellung so unüblich?
Ich würde mich über eine kurze Orientierung freuen, was ich genauer unter die Lupe nehmen oder was ich ändern sollte? @Falco Momentan stecken wir ziemlich fest…
Falco
22. Juni 2021 um 16:55
3
Hast du die Seiteneinstellung secure_uploads aktiviert? Das funktioniert mit MinIO überhaupt nicht .
Falco
22. Juni 2021 um 21:01
5
Ist die S3-Konfiguration gemäß der Dokumentation unter Verwendung von Object Storage für Uploads (S3 & Clones) eingerichtet?
dirien
23. Juni 2021 um 04:44
6
Ja, wirklich Buchstabe für Buchstabe
dirien
1. Juli 2021 um 05:03
7
Entschuldigung für die Störung, aber könntest du etwas entdecken?
dirien
8. Juli 2021 um 05:32
8
Hi, hast du schon eine Idee? Entschuldige bitte die Störung…
Hallo,
ich habe heute auf 2.8.0.beta2 aktualisiert, derselbe Fehler:
Wähle einfach „secure_media
Falco
8. Juli 2021 um 15:50
10
@pmusaraj / @martin sieht nach einer Regression aus, die durch
master ← issue/update-upload-secure-status-on-post-revision
merged 03:32AM - 21 May 21 UTC
When uploads are created from the composer (editing or creating a post),
for si… tes with secure uploads enabled we assume security by default and
that new upload is set to secure. When the post is created, we then
check whether the post uploads _actually_ need to be secure and adjust
accordingly.
We were not doing this when revising a post, so when a new upload was
created when editing a post in a public topic, the secure status stayed
true erroneously causing issues with image previews, among other things.
verursacht wurde, da dabei nicht dieselben Aspekte berücksichtigt wurden, die durch
committed 04:20PM - 25 Nov 19 UTC
In `post_creator`, the ACL update is only necessary when uploads need to be secu… red.
This should fix a regression with S3 clones that do not support updating ACLs.
behoben wurden.
Also macht post_creator:
und models/post:
"topic_tag_changed.added_and_removed",
added: tag_list_to_raw(added),
removed: tag_list_to_raw(removed),
)
elsif added.present?
I18n.t("topic_tag_changed.added", added: tag_list_to_raw(added))
elsif removed.present?
I18n.t("topic_tag_changed.removed", removed: tag_list_to_raw(removed))
end
end
def self.tag_list_to_raw(tag_list)
tag_list.sort.map { |tag_name| "##{tag_name}" }.join(", ")
end
# Revises a post with the given fields and options.
#
# @param editor [User] The user performing the revision
# @param fields [Hash] Hash of fields to update
# @param opts [Hash] Optional parameters for the revision
# @option opts [Time] :revised_at Changes the date of the revision
1 „Gefällt mir“
martin
9. Juli 2021 um 00:18
12
Es gab bereits eine Korrektur, die aus Leistungsgründen durchgeführt wurde, aber auch dein Problem lösen sollte:
committed 04:15PM - 21 Jun 21 UTC
When secure uploads are enabled, editing a post with many uploads can
cause a t… imeout because the store has to be contacted for each upload.
Obwohl ich den Job so anpassen werde, dass er frühzeitig beendet wird, wenn sichere Medien deaktiviert sind.
Tatsächlich kann ich das oben Genannte nicht umsetzen, da wir möchten, dass hochgeladene Anhänge als nicht sicher markiert werden, wenn sichere Medien deaktiviert sind oder wenn der Beitrag in eine andere Kategorie verschoben wurde. Stattdessen werde ich eine Korrektur hinzufügen, um den Fehler Aws::S3::Errors::NotImplemented abzufangen, wenn wir Discourse.store.update_upload_ACL(self) ausführen. Auf diese Weise sollten Clients, die private ACLs nicht unterstützen, keine Probleme haben.
Edit: Ich habe diese Korrektur heute zusammengeführt, hoffentlich hilft sie. FIX: Handle storage providers not implementing ACLs by martin-brennan · Pull Request #13675 · discourse/discourse · GitHub . Ich denke, wir werden nächste Woche auch eine neue Beta-Version veröffentlichen.
3 „Gefällt mir“
Falco
12. Juli 2021 um 11:00
13
Dieses Thema wurde automatisch nach 3 Tagen geschlossen. Neue Antworten sind nicht mehr erlaubt.
