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

fix(scheduler): consider 4xx status code because server works #51

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

gilsondev
Copy link
Owner

Descrição

Este pull request tem como objetivo de aplicar uma correção no que considera um domínio com servidor disponível. Respostas do tipo 4xx podem ser consideradas disponíveis, porque um servidor respondeu o cliente, mesmo quando o recurso solicitado não existe (404) ou não tem autorização para solicitar (403).

Alterações realizadas

  • scheduler/src/ping.py: Aumenta o range das classes de status code, o que forem menores a 5xx
  • scheduler/tests/test_ping.py: Refatoração do teste para cobrir esse caso

Contexto adicional

  • Isso foi necessário porque tem portais do governo que respondem com classes de status antes considerado indisponível, mas que ainda assim buscam redirecionar para outro domínio, como é o caso do amapa.gov.br

Checklist

  • Os testes foram executados e passaram com sucesso
  • A documentação foi atualizada, se necessário
  • O código segue as diretrizes de estilo do projeto
  • Foram adicionados testes automatizados para as novas funcionalidades ou correções

@gilsondev gilsondev added the backend:scheduler Issues relacionadas ao módulo scheduler label Oct 14, 2023
@gilsondev gilsondev self-assigned this Oct 14, 2023
@vercel
Copy link

vercel bot commented Oct 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
checkstatus-govbr ✅ Ready (Inspect) Visit Preview Oct 14, 2023 11:58am

@gilsondev gilsondev merged commit 2f76f6e into main Oct 14, 2023
@gilsondev gilsondev deleted the fix-ping branch October 14, 2023 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:scheduler Issues relacionadas ao módulo scheduler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant