@zogstrip Hast du kurz Zeit, diesen PR zu prüfen, da du Kontext von deiner letzten Prüfung in diesem Bereich hast? FIX: Make migrations from S3 more robust; fix bare URL migration by johnsonm · Pull Request #10093 · discourse/discourse · GitHub
Ich habe darin die Fixes enthalten, die ich während dieser relativ großen Migration durchgeführt habe. Ich habe nicht versucht, für jeden Fix Tests hinzuzufügen; ich bin mir nicht sicher, wie man jede Form von Fehler injizieren kann. Aber zumindest ist die neue Funktionalität getestet.
@RGJ Ich denke, dass mein PR in seinem aktuellen Zustand alle deine ersten beiden Punkte außer dem CDN-Aspekt abdecken könnte. Ich bin mir jedoch beim CDN nicht sicher. Meine Seite nutzte ein CDN und migrierte Videos, die CDN-URLs hatten, aber das könnte ein Nebeneffekt der Namensgebung mit Discourse Spaces gewesen sein. Falls du weitere Fälle hast, wird mein PR hoffentlich eine einfache Grundstruktur bieten, um die Regex und Testfälle für zusätzliche Varianten hinzuzufügen.
Ich denke, es ist richtig, zuerst nach Posts zu migrieren, da nach dem Migrieren der Uploads in einem Post dieser neu gerendert (rebaked) werden muss, damit der gerenderte Post die richtigen URLs enthält. Nachdem ich meine Posts migriert habe (was jetzt, da ich meine Rate-Limiting-Logik geändert habe, um direkt die Queue-Länge zu prüfen, weniger als zwei Wochen dauern könnte), werde ich mich um die verbleibenden Aufräumarbeiten kümmern.
Da mehrere Posts Referenzen auf denselben Inhalt teilen können, falls mehr als eine Person dieselbe Datei hochlädt, ist ein zweiter Durchgang erforderlich, der die gerenderten Daten auf alte URLs prüft und diese Posts neu rendert, um die neuen Speicherorte zu übernehmen. Dabei kann dieselbe Rate-Limiting-Logik verwendet werden, um Queues nicht zu überlasten.
Bei makerforums werde ich wahrscheinlich keine defekten Logos sehen, da wir das Branding angepasst haben, nachdem wir keine neuen Inhalte mehr in „s3