[FIX] Problema na emissão da NFC-e

[corredato]

A emissão de NFC-e estava com um problema:

O método analisar_retorno_raw possui uma variável 'match' que recebe o valor de uma busca no xml através de regex, porém, o 'soap' do regex não existe no xml, e sim 'env'

[antoniospneto]

@corredato consegue arrumar a versão da nfelib para os testes rodarem e conferir se vai passar em tudo?

[mileo]

nfelib

A nfelib não aceita todas as versões do python mais.

[mileo]

@corredato pode colocar algo explicativo log caso o match não obtenha algo, para nunca mais cairmos nesse problema de ficar horas procurando o problema.

[antoniospneto]

nfelib

A nfelib não aceita todas as versões do python mais.

Mas ai podemos remover as versões do python não suportadas dos testes, precisamos ver se os testes estão passando pra saber se houve alguma regressão.

[antoniospneto]

é bom verificar se os testes estão passando, para evitar regressão.

[rvalyi]

eu removi o Python 3.7 recentemente da nfelib pois ele ja ta end of life faz mais de 1 ano (e agora vem o EOL do 3.8 alias)...
Se for um problema pode usar a nfelib 2.0.7 ainda, mas as lib downstream precisam evoluir tb, nao da para lib downstream obrigar o upstream a travar no tempo...

[antoniospneto]

@corredato @mileo

Tenho receio de que essa alteração possa quebrar a comunicação com outros webservices. Analisando melhor, acho que precisamos tornar o regex mais genérico para lidar com os diferentes prefixos que podem aparecer no body.

Sugiro utilizar algo mais genérico como:

pattern = r'<[a-zA-Z0-9:]*Body.*?>(.*?)</[a-zA-Z0-9:]*Body>
match = re.search(pattern, retorno.text.replace("\n", ""))

Acredito que essa abordagem seja mais robusta e evitará problemas com prefixos variados nos webservices. Além disso, outras pessoas já enfrentaram problemas semelhantes, como podemos ver nas seguintes referências:

• Erro ao validar NF-e #55
• Emissão NFe-MS apresenta problemas no SOAP, envio e resposta. #47
• [FIX] Solução de contorno para Bahia e Paraná #50

[corredato]

@antoniospneto Testei a emissão de nfce e nfe com esse pattern genérico e funcionou, já commitei, obrigado.

[antoniospneto]

@corredato
Conseguer verificar o pre-commit que não tá passando? tbm arrumar os testes? tem que travar a nfelib na versão 2.0.7

pode analisar novamente?

[antoniospneto]

@mileo vc tinha marcado como rascunho, tá tudo certo agora?

[mileo]

@mileo vc tinha marcado como rascunho, tá tudo certo agora?

Sim! Agora tá ok!