BrasilAPI / cep-promise

Busca por CEP integrado diretamente aos serviços dos Correios, ViaCEP e outros (Node.js e Browser)
MIT License
2.88k stars 316 forks source link

Muitos erros no console. #121

Closed klawdyo closed 4 years ago

klawdyo commented 6 years ago

Quando uma requisição retorna falha de algum dos serviços, o console fica cheio de erros.

image

O catch tá pegando legal e só está sendo chamado quando todos estão com erro, conforme previsto. Só gostaria de deixar o console limpo, visto que não é muito profissional colocar em produção um produto cheio de erros aparecendo pro usuário.

Poderia ter uma opção para desabilitar esses logs em produção?

lucianopf commented 6 years ago

Bom dia @klawdyo esse é um problema que eu venho tentando atacar por algum tempo mas não é tão trivial quanto parece =/

Existem algumas razões pro request dar erro, algumas delas eu consigo ver no seu console que parecem fáceis de consertar, por exemplo o uso do mesmo protocolo (https ou http) do que a origin.

Alguns não tanto, como por exemplo quando um dos serviços realmente retornam um erro, e portanto isso não mais é solucionável via lib mas sim na API do service que estamos usando.

O problema é que usamos o fetch do browser (quando disponível) e o comportamento padrão dele é para alguns erros jogar no console de forma intratável, tenta por exemplo executar o seguinte código:

fetch('https://github.com/filipedeschamps/cep-promise/issues/121/404').then(console.log).catch(console.log)

Que resultado, MESMO tratando no catch há um console.error no navegador: screen shot 2018-09-03 at 9 48 57 am

Eu acho isso bizarro também, mas é o comportamento nativo do fetch já embarcado nos navegadores =S

De qualquer forma, continuo buscando soluções pra esse maldito console.error que também me incomoda HORRORES! 😂 👍

(E aceito qualquer dica que tiver sobre o assunto!!! hehehe)

lucianopf commented 6 years ago

Mais uma ref triste =/

brunooomelo commented 5 years ago

fiz um teste simples com console.clear, acho que funcionou.

fetch('https://github.com/filipedeschamps/cep-promise/issues/121/404')
  .then(console.clear)
  .catch(console.clear)

ezgif com-video-to-gif

brunooomelo commented 5 years ago

fiz outros testes aqui, e funcionou bem, o problema que aparece "Console was cleared". @lucianopf

lucianopf commented 5 years ago

O problema é que limpa o console, o que não necessariamente é o desejado por quem ta desenvolvendo =/

Mas muito obrigado por lembrar disso @brunooomelo !! 🤓 👏

quinhone commented 5 years ago

https://proxier.now.sh/http://www.cepaberto.com/api/v2/ceps.json?cep=78095390 some errors in console:

quinhone commented 5 years ago

warning: blocked cross-origin response https://viacep.com.br

lucianopf commented 5 years ago

Hum, o do proxier me parece estar ligado com a requisição do cepaberto 🤔 Vou dar uma olhadinha melhor =D

Misericórdia, o viacep começou a bloquear CORS agora, vou colocar ele no proxy então tbm 🤦‍♂

lucianopf commented 5 years ago

Cepaberto:

Screen Shot 2019-04-29 at 18 20 10

@filipedeschamps precisamos pensar melhor nesse service, talvez desliga-lo, o que acha? 🤔

filipedeschamps commented 5 years ago

Vish!! Eu topo total 👍

lucianopf commented 5 years ago

Muito obrigado @quinhone , abri um PR para desabilitar o serviço problemático e injetar o proxy que burla as regras de CORS pra prosseguirmos utilizando o service ViaCep que ao meu ver é o mais completinho de todos 😬

quinhone commented 5 years ago

Cepaberto:

Screen Shot 2019-04-29 at 18 20 10

@filipedeschamps precisamos pensar melhor nesse service, talvez desliga-lo, o que acha? 🤔

Apoio, ele está dando muito erro 500

filipedeschamps commented 4 years ago

Pessoal, o @lucianopf fez um fix que está na mais recente versão: cep-promise@3.0.8

image

filipedeschamps commented 4 years ago

Pessoal, estou fechando essa issue após o lançamento do cep-promise@3.0.9 mas qualquer coisa sintam-se livres em reabrir ela caso o problema persista 👍