@RGJ Ich habe das überhaupt nicht geändert; das Neu-Erstellen von Upload war eine Fehlerbehebung von @zogstrip unter Cannot execute the rake uploads:migrate_from_s3 - #11
Ich versuche lediglich, den bereits vorhandenen Code zum Laufen zu bringen, und kenne die Discourse-Interna nicht gut; ich habe mich hier oft im Dunkeln herumgetastet. Meine einzige Ruby-Erfahrung sind die wenigen PRs, die ich für Discourse eingereicht habe. Dem Muster des bestehenden Codes zu folgen scheint nicht der effizienteste Weg zu sein (siehe meine Diskussion mit @pfaffman oben zum Thema Short-Circuiting); dem stimme ich voll zu. Wie du an der Tatsache erkennen kannst, dass ich heute Morgen noch nicht einmal bemerkt habe, dass OptimizedImages.url ebenfalls auf einen /uploads-Pfad geändert und etag auf nil gesetzt werden muss (und was noch alles), bin ich immer noch am Blindflug.
Ich muss weiterhin zumindest zuerst durch die Beiträge iterieren, um alte, direkte URLs in Beiträgen zu korrigieren. Weitere Korrekturen sind noch nötig, wie etwa das Nicht-Neu-Validieren von Beiträgen und das Nicht-schweigende Unterdrücken von Fehlern. Ich möchte weiterhin ein Rate-Limit einsetzen, um die Auswirkungen auf Live-Systeme zu begrenzen.
Zu deinen ersten beiden Bedenken im Zusammenhang mit Nicht-Post-Updates: Hier ist meine Arbeit im Gange, noch nicht auf einem Live-System getestet (Commit), die vielleicht hilfreich sein könnte. Ich werde sie jedoch erst auf meinem Live-System testen, wenn die Migration der Post-Uploads abgeschlossen ist.
Das Anpassen von etwas, das früher funktionierte, war alles, wozu ich gerade Lust hatte. Wenn du eine schnellere Migration durchführen möchtest, bin ich dafür; es könnte Sinn ergeben, meine langsamere – aber zumindest bessere – Arbeit als Pareto-Verbesserung zu mergen, und dann könntest du sie komplett durch etwas deutlich Besseres ersetzen. Ich wäre der Erste, der das feiert, selbst wenn ich zu diesem Zeitpunkt nicht mehr in der Lage sein sollte, es zu nutzen.