Set user-agent error on Browser

[ivancorrea]

Ao tentar alterar o user-agent, o browser acusa o erro: Refused to set unsafe header "user-agent"

[coveralls]

Coverage remained the same at 100.0% when pulling a554a75 on ivancorrea:master into 9bca1c4 on filipedeschamps:master.

[filipedeschamps]

Muito obrigado pela contribuição 😍 uma sugestão: sempre enviar o user-agent identificando o “cep-promise”, tanto no back como no front, isto vai ser mais justo com quem quiser nos bloquear 🤝

…

On Sun, Sep 27, 2020 at 12:33 PM Ivan Correa ***@***.***> wrote: Ao tentar alterar o user-agent, o browser acusa o erro: Refused to set unsafe header "user-agent" ------------------------------ You can view, comment on, or merge this pull request online at: https://github.com/filipedeschamps/cep-promise/pull/189 Commit Summary - viacep: browser user-agent error - format File Changes - *M* src/services/viacep.js <https://github.com/filipedeschamps/cep-promise/pull/189/files#diff-cac76ab3f5e516e62610c849df417a44> (7) Patch Links: - https://github.com/filipedeschamps/cep-promise/pull/189.patch - https://github.com/filipedeschamps/cep-promise/pull/189.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <https://github.com/filipedeschamps/cep-promise/pull/189>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABANEEJGKQLWXUTJQ7MX7QDSH6HPPANCNFSM4R3XPYGA> .

[ivancorrea]

Concordo @filipedeschamps !

Apenas utilizei o código do commit que causou o erro.

Já atualizei o PR identificando o UA

[hexetia]

Bem que essa pr podia entra na próxima release, toda vez que o cep-promise faz uma requisição no browser uma mensagem de erro aparece no console reportando que não é possível mudar o user-agent por motivos de segurança

@ivancorrea você fazer outro commit nessa pr pra mudar somente o trecho de código que causa o erro?
pq tem "outras alterações" que só removem o espaço em branco antes do parâmetro das funções

[ivancorrea]

@morhogg

Acredito que este PR pode morrer aqui mesmo, visto que o @LorhanSohaky fez um PR para permitir a configuração do User Agent no PR #191

[lucianopf]

É um ótimo ponto @ivancorrea como seria configurado dinamicamente acho que resolve o mesmo problema desse PR 🤔

Mestres, só pra não deixar vcs perdidos abri uma issue com um planinho de ação pra reorganizar o repo dado a migração pra org do BrasilAPI 😬

#197

Apesar de ser uma ideia interessante pelo que entendi é inviável =/

[lucianopf]

Mestre @ivancorrea muuuito obrigado pela resolução!! ❤️ 🚀

Fiz o merge e publiquei a versão 4.0.3 e parece ter funcionado! 🤩

MUUUITO obrigado pela ajuda e perdão a demora para resolução, à partir dessa semana (após a black friday) planejo dedicar mais tempo para projetos OSS 😬 🙏