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

feat(router): add OPTIONS method for route handling #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adeilsonaalima
Copy link

@adeilsonaalima adeilsonaalima commented Nov 20, 2023

Em algumas situações o navegador precisa enviar uma requisição preflight de CORS, de método OPTIONS, que verifica se o protocolo CORS é entendido e se o servidor aguarda o método e cabeçalhos especificados. No meu caso concreto, um preflight retornava um 200 OK ou 302 Found (se ligado o redirect de erro) mas não completava a "2a" solicitação do navegador.

Identifiquei que a causa do problema era a falta de um rota OPTIONS homônima a da minha requisição ou um tratamento anterior a isso... então resolvi com um simples if SERVER_METHOD === OPTIONS que passava os cabeçalhos e dava exit, mas também testei a solução que propus no commit.

Não sou experiente, o método sugerido é mais para exemplificar a situação.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant