[14.0][REF] Geração do Certificado Fake, removido dependência do PyOpenSSL. #1778
Conversation
|
ping @netosjb.. acho que vc já viu |
Pois é foi exactamente por isso que eu tinha comentado a geração dos certificados fakes para a gente pode fazer o merge do l10n_br_fiscal. Isso seria uma boa arrumar esse problema no erpbrasil.assinatura... |
|
pois é sim, essa pr ainda fica pendente o reparo na erpbrasil.assinatura :/ |
Abri uma PR trivial no erpbrasil.assinatura corrigindo esse warning: erpbrasil/erpbrasil.assinatura#28 @OCA/local-brazil-maintainers |
|
Pessoal, eu comentei o PR erpbrasil/erpbrasil.assinatura#26 e erpbrasil/erpbrasil.assinatura#28 |
| @@ -74,36 +82,63 @@ def create_fake_certificate_file(valid, passwd, issuer, country, subject): | |||
| :param subject: Some string: CERTIFICADO VALIDO TESTE | |||
| :return: base64 file | |||
| """ | |||
| key = crypto.PKey() | |||
| key.generate_key(crypto.TYPE_RSA, 2048) | |||
| private_key = rsa.generate_private_key( | |||
There was a problem hiding this comment.
Eu havia comentado um tempo atrás que o ideal era portar essa funcionalidade para a lib erpbrasil.assinatura até foi colocado na docstring do método create_fake_certificate_file então eu diria que após a aprovação do PR erpbrasil/erpbrasil.assinatura#26 esse método poderia ser portado e removido os commits de mudança neste arquivo neste PR...
There was a problem hiding this comment.
Concordo Renato, muito melhor se a migração desse código para a lib já acontecer já.. vamos esperar o pessoal aprovar lá então.
| @@ -1,9 +1,9 @@ | |||
| # generated from manifests external_dependencies | |||
| cryptography | |||
There was a problem hiding this comment.
Fazendo o porte do método create_fake_certificate_file para a erpbrasil.assinatura não precisa especificar essa dependência
|
@netosjb vc conseguiria fazer esse refactor num PR para https://github.com/erpbrasil/erpbrasil.assinatura como o @renatonlima sugeriu? Renato e eu a gente tem a permissão no repo erpbrasil.assinatura. O que nao temos é a conta no pypi, mas a gente consegue botar o pacote numa outra conta com nome alternativo de boa ate o @mileo resolver isso. Nisso seria bom ter esse refactor la que a gente já poderia ter esse certificado fake para testar melhor a NFe na v14... |
|
@rvalyi deixa comigo, vou abrir a pr lá |
|
@netosjb se puder fazer um rebase eu agradeço. |
|
@netosjb se puder faça o rebase desta tbm fazendo fazendo favor. |
antoniospneto
force-pushed
the
14.0-ref-certicates
branch
from
d4b979e
to
063a398
Compare
|
substituído por #1843 |
O mesmo que #1777
Porém tem 2 dois commits adicionais:
reativei o código comentando que insere os certificado fake nas empresas de demostração. c56815f
Inclui um fix para exibição do nome do arquivo do certificado d4b979e
vou fazer um back-port desse commit para a v12 também.
Obs: lembrando que ainda precisar ser aprovado o merge no erp.assinatura para remoção dos warning e melhorar compatibilidade das libs com o trevis: erpbrasil/erpbrasil.assinatura#26