search

lealhugui / node-dfe

Geração/integração de NF-e/NFC-e para node.js
MIT License
257 stars 99 forks source link

corrige o erro: UNABLE_TO_GET_ISSUER_CERT_LOCALLY #19

Closed tfiliano closed 4 years ago

tfiliano commented 4 years ago

18

lealhugui commented 4 years ago

Excelente proposta @tfiliano , eu apenas fiz algumas consideracoes, que devem ser sanadas.

tfiliano commented 4 years ago

Essas configuracoes de certificado mudam conforme o tipo de certificado original e a cadeia de certificados. É melhor que essa seja uma configuracao "opt in".

Mude para um parametro que possa ser informado pela app cliente.

Estou providenciando.

Guilherme, quais características de certificado você utilizou para funcionar sem esses parâmetros? Estou considerando migrar aqui pro axios, porque o request esta deprecated. Vou realizar alguns testes, e subo pull request, com essas alterações e a parametrização que você pediu.

lealhugui commented 4 years ago

Estou providenciando.

Guilherme, quais características de certificado você utilizou para funcionar sem esses parâmetros? Estou considerando migrar aqui pro axios, porque o request esta deprecated. Vou realizar alguns testes, e subo pull request, com essas alterações e a parametrização que você pediu.

Usei um .px, que foi convertido para .pem, e depois exportada a chave privada.

Quanto a mudança pra axios, apenas deve ser validado se vai funcionar pra mais de um ambiente da SEFAZ. Tivemos inúmeros problemas com relação a isso em algumas versões.

E nao se esqueça de fazer isso em PRs separadas, pois de outra forma, nao temos como avaliar separadamente.

Happy Coding ;)

sonarcloud[bot] commented 4 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication