Set user-agent error on Browser #189
Conversation
|
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>
.
|
|
Concordo @filipedeschamps ! Apenas utilizei o código do commit que causou o erro. Já atualizei o PR identificando o UA |
|
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? |
|
@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 |
|
É 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 😬 |
lucianopf
dismissed
filipedeschamps’s stale review
Apesar de ser uma ideia interessante pelo que entendi é inviável =/
|
Mestre @ivancorrea muuuito obrigado pela resolução!! ❤️ 🚀 Fiz o merge e publiquei a versão 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 😬 🙏
|
Ao tentar alterar o user-agent, o browser acusa o erro: Refused to set unsafe header "user-agent"