Quando preview de um asset é muito grande (>4MiB) backend dá erro 500

gabrielrra commented 1 year ago

Mandatory informations:

Are there any customers directly impacted by this bug? Which ones?

Todos os customers que tenham um preview muito grande de um asset.

Bug Category

[ ] Connection Manager
[ ] Connection Test
[ ] Creating Pipeline
[ ] Executing Pipeline
[x] Cataloging Data Asset
[ ] Other

Describe the bug

Ao buscar o preview de um asset a API retorna erro 500:

To Reproduce

Steps to reproduce the behavior:

Acessar https://app.stg.dadosfera.ai/pt-BR/collect/import-files/e37af5ab-3a2f-4f0d-a081-856db2600bea
Inspecionar

Expected behavior

Mostrar uma mensagem avisando que o preview não está disponível por ser muito grande ao invés de mensagem de erro.

Screenshots

Does this bug impact any demo or a sale?

Ainda não

Dadosfera Customer:

dadosfera

Other informations:

Any logs, error output, etc?

Resposta da API (GET http://localhost:3333/catalog/data-asset/6803d515-c003-4e80-9405-1d8d05873dae/preview)

{
  "statusCode": 500,
  "message": "Ocorreu um erro desconhecido. Caso o erro persista, entre em contato com o suporte",
  "error": "Oops... Não foi possível realizar sua ação",
  "code": "UNKNOWN",
  "details": "Received message larger than max (4677005 vs. 4194304)"
}

What environment of software are you using?

[ ] PRD
[x] stg

Browser Engine and Version: Google Chrome Version 110.0.5481.100 (Official Build) (64-bit)

When the bug happened: 23/02/2022 10:59

Colombo97 commented 1 year ago

@gabrielrra, O front aparentemente ta refletindo oque o back manda, será que não deveria vir esse erro na prop message ao invés de no details?

gabrielrra commented 1 year ago

Sim, vamos tratar o erro no backend para retornar a mensagem de erro traduzida

gabrielrra commented 1 year ago

O que eu pensei também era evitar de abrir popup de erro e não ter o texto vermelho na tela nesse caso, colocar alguma coisa mais amigável. Pelo menos até decidirmos como lidaremos com essas respostas maiores.

Colombo97 commented 1 year ago

Saquei, vou preparar aqui então.

@gabrielrra, vou precisar de algum tipo de flag para esta situação para poder bypassar o comportamento padrão de exibir o toast de erro e a mensagem no preview do data asset (Fazer uma diferenciação de erro).

Colombo97 commented 1 year ago

Novas flags de diferenciação: CATALOG.PREVIEW_TOO_BIG CATALOG.METADATA_TOO_BIG

dadosfera / Bugsfera