Problemas com Migrate_from_s3

@RGJ Eu não mudei isso em nada; recriar o Upload foi uma correção de bug feita por @zogstrip em Cannot execute the rake uploads:migrate_from_s3 - #11

Estou apenas tentando fazer funcionar o código que já existia, e não conheço bem os detalhes internos do Discourse; tenho estado às cegas por bastante tempo. Minha única experiência com Ruby são os poucos PRs que fiz para o Discourse. Seguir o padrão do código existente parece não ser a rota mais eficiente (veja minha conversa com @pfaffman acima sobre curto-circuitar), concordo totalmente. Como você pode notar pelo fato de que, mais cedo hoje, eu nem percebi que OptimizedImages.url também precisaria ser modificado para um caminho /uploads e que o etag precisaria ser definido como nil (e não sei mais o quê), ainda estou voando às cegas.

Ainda preciso iterar sobre as postagens primeiro, pelo menos para corrigir URLs literais antigas nas postagens. Ainda preciso de algumas das outras correções, como não revalidar postagens e não engolir erros silenciosamente. Ainda acho que preciso implementar limitação de taxa, para limitar o impacto em sites em produção.

Quanto às suas duas primeiras preocupações envolvendo atualizações de não-postagens, aqui está meu trabalho em andamento, ainda não testado em um site em produção (commit) que pode ajudar, mas que só vou testar no meu site em produção quando toda a migração de uploads de postagens estiver concluída.

Ajustar algo que já funcionava era tudo o que eu tinha disposição para fazer. Se você quiser fazer uma migração mais rápida, eu sou totalmente a favor; pode fazer sentido mesclar meu trabalho mais lento, mas pelo menos melhor, como uma melhoria de Pareto, e então você poderia substituí-lo completamente por algo muito melhor, e eu seria o primeiro a celebrar, mesmo que eu não esteja mais em posição de usá-lo naquele momento.

1 curtida