@RGJ Je n’ai rien modifié à cela ; la recréation d’Upload était une correction de bug apportée par @zogstrip dans Cannot execute the rake uploads:migrate_from_s3 - #11
Je tente simplement de faire fonctionner le code déjà présent, et je ne connais pas bien les internals de Discourse ; j’ai beaucoup tâtonné dans le noir. Ma seule expérience en Ruby provient des quelques PR que j’ai réalisées pour Discourse. Suivre le modèle du code existant semble effectivement ne pas être la voie la plus efficace (voir ma conversation avec @pfaffman plus haut concernant la coupure courte), je suis tout à fait d’accord. Comme vous pouvez le constater, vu que plus tôt aujourd’hui je n’avais même pas réalisé que OptimizedImages.url devrait également être modifié pour un chemin /uploads et que l’etag devrait être défini sur nil (et je ne sais pas quoi d’autre), je continue d’agir à l’aveugle.
Il faut toujours parcourir les publications en premier, au moins pour corriger les anciennes URLs littérales dans les posts. Il reste encore d’autres corrections à apporter, comme ne pas revalider les posts et ne pas avaler silencieusement les erreurs. Je pense toujours qu’il faut limiter le débit pour réduire l’impact sur les sites en production.
Pour vos deux premières préoccupations concernant les mises à jour non liées aux posts, voici mon travail en cours, pas encore testé sur un site en production (commit) qui pourrait aider, mais que je ne testerai sur mon site en production qu’une fois la migration des uploads de posts terminée.
Ajuster quelque chose qui fonctionnait auparavant était tout ce que j’avais envie de faire. Si vous souhaitez effectuer une migration plus rapide, je suis tout à fait favorable ; il pourrait être judicieux de fusionner mon travail plus lent mais au moins meilleur en tant qu’amélioration de Pareto, puis vous pourriez le remplacer entièrement par quelque chose de bien meilleur, et je serai le premier à célébrer, même si je ne serai plus en mesure de l’utiliser à ce moment-là.