cep-promise
cep-promise copied to clipboard
BrasilAPI
Configurações da biblioteca e adição de keep-alive
Motivação
Dependendo da quantidade de requisições feitas, perde-se muito tempo fazendo o handshaking e isso pode ser facilmente resolvido com o cabeçalho keep-alive. A implementação foi baseada na issue#423 do node-fetch.
Implementação
Quando estava implementando fiquei na dúvida se era uma boa escolha colocar isso como default ou uma configuração da lib, então para evitar que essa modificação cause algum problema para os demais desenvolvedores preferi passar como uma configuração. E com isso aproveitei para possibilitar que as proxies também sejam passadas como configuração aos serviços.
Por fim, agora caso queiramos passar qualquer tipo de configuração aos serviços não precisaremos mais (talvez / espero eu hahaha) modificar a interface.
Observações
- Não sabia como poderia testar se a requisição voltou com keep-alive na resposta sem afetar a interface do cep-promise
- Um dos testes falhou, mas creio que não está relacionado com as mudanças que fiz
- Fiz algumas correções de code-style
- Se forçar code-style for algo muito importante, é preciso modificar o
lint-fixpara algo assim:"lint-fix": "standard '*.js' 'src/**/*.js' 'test/**/*.js' --fix",
Coverage remained the same at 100.0% when pulling e2bb7128bc0493e3fff369ec01db7faa6fce1d13 on LorhanSohaky:feature/keep-alive into 9bca1c4aec436667fcee7d7c837f5c8b2b752a6f on filipedeschamps:master.
Coverage decreased (-0.6%) to 99.419% when pulling 5f060130ad079de87b294f662eb70e1018490571 on LorhanSohaky:feature/keep-alive into 9bca1c4aec436667fcee7d7c837f5c8b2b752a6f on filipedeschamps:master.
Vi agora na Newsletter do Filipe sobre o Hacktoberfest. Acham possível eleger esse PR? Assim poderei participar com este 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 😬
https://github.com/BrasilAPI/cep-promise/issues/197
