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

Dúvidas a respeito das violações específicas do endpoint GET /cobv #316

Closed
marcosf opened this issue Feb 9, 2021 · 3 comments
Closed
Assignees
Labels
Documentação Issues relacionados à documentação, versões, melhorias de texto, etc... melhoria algo está funcionando, mas poderia ser melhor

Comments

@marcosf
Copy link

marcosf commented Feb 9, 2021

Tenho uma dúvida a respeito de duas violações do endpoint GET /cobv, que são estas violações:

  • o parâmetro paginacao.paginaAtual indica uma página fora dos limites da atual consulta.
  • o parâmetro loteCobVId não corresponde a um lote existente.

Para poder validar as mesmas eu teria que fazer pelo menos dois hits a mais no banco de dados a fim de calcular se a pagina enviada pelo requisitante existe(se não excede o número total de páginas) e para o lote de cobrança tenho que buscar se o id do lote existe. Estas buscas podem inviabilizar requisitos de performance de api, sem contar que teremos acessos ao banco de dados que poderiam ser evitados.

Poderíamos ignorar estas validações? Pois se a pagina for inválida, poderemos enviar um array vazio como response. No caso do lote seria a mesma coisa.

@ninrod ninrod self-assigned this Feb 10, 2021
@ninrod ninrod added Documentação Issues relacionados à documentação, versões, melhorias de texto, etc... melhoria algo está funcionando, mas poderia ser melhor labels Feb 10, 2021
@ninrod
Copy link
Member

ninrod commented Feb 10, 2021

bom dia @marcosf

Poderíamos ignorar estas validações? Pois se a pagina for inválida, poderemos enviar um array vazio como response. No caso do lote seria a mesma coisa.

Concordo. Na 2.2.1 elas saem.

@marcosf
Copy link
Author

marcosf commented Feb 10, 2021

@ninrod Muito Obrigado pela resposta.

@ninrod
Copy link
Member

ninrod commented Feb 10, 2021

Já no master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentação Issues relacionados à documentação, versões, melhorias de texto, etc... melhoria algo está funcionando, mas poderia ser melhor
Projects
None yet
Development

No branches or pull requests

2 participants