Transparencia-Brasil / achados-e-pedidos-site

0 stars 1 forks source link

[bugfix] Erro com upload de arquivos (fluxo de usuário) #66

Closed rdurl0 closed 1 year ago

rdurl0 commented 1 year ago

Upload de anexos falham

⚠️ erro 500

1- Nesse pedido falta o arquivo Pedido DSEIY - Atendimentos e remoções - Dados remoções 2021.xlsx (4,6MB) na primeira resposta do órgão, onde estão outros 4 arquivos.

2- Nesse pedido falta o Malaria_serie_Historica.xlsx (7,2MB) na última resposta.

Os arquivos estão aqui

O anexo não é registrado no banco de dados, exemplo de consulta:

SELECT * FROM pedidos WHERE Slug = "dsei-yanomami";
select * from pedidos_interacoes where CodigoPedido = "294112";
select * from pedidos_anexos where CodigoPedidoInteracao in ("278242","278243", "278244");

Destino de anexo inesperado

No bucket, os anexos que carregaram foi para uma pasta inesperada. O correto não seria guardar o arquivo em no diretório da CGU

achados/pedidos/CGU/PedidoDSEIYAtendimentoseremoesDadosremoes2020278242_1.xlsx

Os anexos estão guardados na pasta

achados/pedidos/PedidoDSEIYAtendimentoseremoesDadosremoes2020278242_1.xlsx
studiocuboweb commented 1 year ago

Oi @rdurl0 nós já temos um problema com upload de arquivos mapeado nesse issue. Se o fluxo de upload dos arquivos for o mesmo do issue que eu passei então creio que os dois possuem relação.

Vcs precisam validar o outro issue no stage para que eu publique o ajuste e ver se resolve essa questão.

Fico no aguardo.

rdurl0 commented 1 year ago

Paulo, o teste que fiz na issue #63 sugere que esses não são problemas relacionados.

rdurl0 commented 1 year ago

Outro problema parecido foi encontrado:

Após subir este pedido apenas 1 dos anexos foram incorporados à Resposta do órgão público.

Os arquivos que deveriam ter subido junto com a resposta estão nesta pasta zipada (descompacte-a para reproduzir o teste).

obs: corrigimos os colchetes no nome do arquivo e refizemos o teste sem sucesso

image image image

studiocuboweb commented 1 year ago

@rdurl0 e @talitalobo

Corrigido o upload de arquivo.

O principal problema era a limitação dos parâmetros do PHP com relação ao tamanho do arquivo. O limite permitido pelo sistema estava em 2 MB e eu configurei para 1500M. Na descrição desta issue os dois arquivos com problema estavam com o tamanho acima dos 2MB...

Outros parâmetros também foram configurados e vcs podem verificar nesse pull request

Além disso, melhorarmos o código e implementamos um log.

Esse sistema de upload não possui um tratamento de erro para esse tipo de situação e por isso os erros são simplesmente ignorados pelo sistema sem dar feedback. Dessa forma, arquivos que não possuem problema são publicados e aqueles que possuem são ignorados dando a impressão de erro.

Caso vocês queiram mudar a rotina de validação e tratamento de erro desses arquivos, vamos criar outro issue para tratar disso oK?

Favor validar no stage. Em produção os parâmetros tb foram alterados e os arquivos maiores de 2MB tb estão sendo publicados (embora não teve a atualização dos ajustes no código).

Para fechar esse ticket:

  1. entrar em produção e no stage
  2. Publicar mais de um arquivo (com tamanhos dentro do novo limite configurado)
  3. Se todos os arquivos subirem e vcs conseguirem fazer o download então eu publico em produção os ajustes
  4. Testar novamente em produção
  5. Se der certo eu fecho esse ticket.

Se aparecer algum novo erro durante o processo, criamos outro issue para tratar.

Vamos fazer dessa forma?

Fico no aguardo.

talitalobo commented 1 year ago

Oi @studiocuboweb , já foram realizadas inserções de arquivos de diversos tamanhos no banco/site (em produção). Considerando que as versões estão sincronizadas, então esta issue já foi validada. Vou adicionar esse comentário lá no trello também.