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

DOS ou alguma coisa estranha aconteceu com o Middleware #744

Closed
filipedeschamps opened this issue Sep 20, 2022 · 4 comments
Closed

DOS ou alguma coisa estranha aconteceu com o Middleware #744

filipedeschamps opened this issue Sep 20, 2022 · 4 comments

Comments

@filipedeschamps
Copy link
Owner

Olha esse gráfico no Upstash:

image

Digo estranho, porque antes disso a latência do serviço aumentou:

image

E ambos os ambientes apresentaram esse pico:

image

E nenhuma status page mostrava nenhuma instabilidade: Vercel, Upstash, AWS e Cloudflare.

O que suspeito é que algo aconteceu na Upstash, onde os logs das lambdas marcavam que passavam de 30 segundos e retornava um erro.

@aprendendofelipe
Copy link
Collaborator

O que suspeito é que algo aconteceu na Upstash, onde os logs das lambdas marcavam que passavam de 30 segundos e retornava um erro.

Será? O Upstash não deveria causar problemas para as lambdas, mas só para a Edge. E na Edge o timeout é de 5s e não 30s.

Talvez algo na AWS que tenha afetado tanto a Vercel quanto o Upstash. O mais estranho é que nesse período o console da Vercel só mostra 6 invocações das lambdas com erro. Nenhuma deu timeout e mais de 1000 deram sucesso. Só que nessas com sucesso o tempo de execução foi lá pra cima.

Talvez só os logs do Upstash e algumas requisições na Vercel tenham sido afetadas. Pois no gráfico da Edge não aparece nenhum pico de acessos.

Sabe se realmente ocorreu alguma instabilidade para quem estava acessando o TabNews? Eu acho que acessei nesse horário e não percebi nada de estranho.

@filipedeschamps, teve um deploy na Vercel nesse mesmo horário. Foi você mesmo?

@filipedeschamps
Copy link
Owner Author

Pois no gráfico da Edge não aparece nenhum pico de acessos.

Justo, e também não aparece nenhum pico nos logs que entram nas lambdas.

Sabe se realmente ocorreu alguma instabilidade para quem estava acessando o TabNews?

Sim, todas as rotas da API não respondiam mais e retornavam 504. Os estáticos continuavam ok.

@filipedeschamps, teve um deploy na Vercel nesse mesmo horário. Foi você mesmo?

Foi, queria entender se isso destravaria algo, mas não ajudou. O interessante é que era possível fazer a geração dos estáticos normalmente, só não era possível conseguir retorno do que passava pelo /api.

E daí o único pico que encontrei foi no Upstash mesmo 👍

@aprendendofelipe
Copy link
Collaborator

Sim, todas as rotas da API não respondiam mais e retornavam 504. Os estáticos continuavam ok.

Isso provavelmente foi só por causa do Upstash mesmo, mas se foi isso, o timeout foi de 5s na Edge.

Estamos tratando erros do Upstash, mas não temos nada para quando ele demorar para responder.

Como tem o limite de 5s da Edge, podemos chamar o NextResponse.next() dentro de setTimeout de uns 4s (ou talvez bem menos), pra eliminar esse ponto crítico que é o Upstash.

@filipedeschamps
Copy link
Owner Author

Estamos tratando erros do Upstash, mas não temos nada para quando ele demorar para responder.

Como tem o limite de 5s da Edge, podemos chamar o NextResponse.next() dentro de setTimeout de uns 4s (ou talvez bem menos), pra eliminar esse ponto crítico que é o Upstash.

Perfeitíssimo! Acabei de adicionar no #493 esse seu comentário na seção de Segurança pra não correr o risco de se perder entre as Milestones 🤝

Vou fechar essa issue mas vou continuar de olho aqui 👍

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

No branches or pull requests

2 participants