@RGJ Non ho modificato nulla del genere; ricreare Upload è stato un fix di bug da parte di @zogstrip in Cannot execute the rake uploads:migrate_from_s3 - #11
Sto solo cercando di far funzionare il codice che era già presente, e non conosco bene gli interni di Discourse; mi sono trovato a barcollare al buio molte volte. La mia unica esperienza con Ruby è costituita dai pochi PR che ho realizzato per Discourse. Seguire il pattern del codice esistente sembra davvero non essere la via più efficiente (vedi la mia conversazione con @pfaffman sopra riguardo all’intercettazione breve), sono completamente d’accordo. Come puoi notare dal fatto che stamattina non avevo nemmeno realizzato che OptimizedImages.url avrebbe dovuto essere modificato anch’esso in un percorso /uploads e che etag dovesse essere impostato su nil (e non so cosa altro), sto ancora volando alla cieca.
Devo ancora iterare sui post, almeno per correggere gli URL letterali vecchi nei post. Devo ancora applicare alcune delle altre correzioni, come non rivalidare i post e non ingoiare silenziosamente gli errori. Penso ancora che sia necessario introdurre un limite di frequenza (rate limiting) per limitare l’impatto sui siti in produzione.
Per le tue prime due preoccupazioni relative agli aggiornamenti non relativi ai post, ecco il mio lavoro in corso, non ancora testato su un sito in produzione (commit) che potrebbe aiutare, ma che non testerò sul mio sito in produzione finché la migrazione degli upload dei post non sarà completata.
Modificare qualcosa che funzionava prima era tutto ciò che avevo voglia di fare. Se desideri eseguire una migrazione più veloce, sono totalmente favorevole; potrebbe avere senso unire il mio lavoro più lento ma almeno migliore come miglioramento di Pareto, e poi potresti sostituirlo completamente con qualcosa di molto migliore. Sarei il primo a festeggiare, anche se in quel momento non sarò più nella posizione di utilizzarlo.