Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timeout definido pelo client #298

Closed
leogregianin opened this issue Oct 20, 2023 · 5 comments
Closed

Timeout definido pelo client #298

leogregianin opened this issue Oct 20, 2023 · 5 comments
Labels
enhancement good-first-issue boa tarefa para iniciante

Comments

@leogregianin
Copy link
Collaborator

Hoje o timeout da requisição ao webservice está predefinido como 300 segundos:

Seria interessante definir como responsabilidade do client conforme sua própria necessidade?
Definir um timeout global ou definir um timeout nos parâmetros de cada serviço?

@juniortada
Copy link
Member

Acredito que o tempo de timeout pode variar entre os tipos de notas (NF-/NFC-e/etc).
Principalmente em casos de NFC-e, a SEFAZ de cada estado tem uma infraestrutura diferente que impacta bastante nesse valor.

Você já teve a necessidade de editar esse valor para algum serviço específico? Para maior ou menor tempo?

@leogregianin
Copy link
Collaborator Author

O timeout varia entre NFe, NFCe e MDFe, entre os tipos de serviço da NFe, principalmente transmissão, distribuição e eventos e também varia entre a SEFAZ de cada estado.
Por isso não sei a melhor forma de tratar isso, acredito que um valor padrão fixo dentro da lib não seja a melhor forma.

@juniortada
Copy link
Member

Concordo, podemos procurar um valor configurado em uma env, separado por tipo de nota (NF-e/NFC-e).
Caso não encontre define o valor default.
Teríamos que disponibilizar um env.sample para convencionar os nomes.

Só uma sugestão, também não tenho uma opinião formada sobre a melhor forma de fazer isso.

@leotada
Copy link
Member

leotada commented Dec 7, 2023

Acho que seria bom passar um valor de timeout nas classes de Comunicacao, no construtor que ja é específico para cada serviço.

@leotada leotada added enhancement good-first-issue boa tarefa para iniciante labels Dec 7, 2023
@juniortada
Copy link
Member

#328

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement good-first-issue boa tarefa para iniciante
Projects
None yet
Development

No branches or pull requests

3 participants