vercel/serve

Make redirections support query strings

Open

#393 aberto em 30 de mai. de 2018

Ver no GitHub
 (6 comments) (5 reactions) (0 assignees)TypeScript (8.811 stars) (719 forks)batch import
help wantedpatch

Description

I've been using version 6.5.8 and I was able to serve URLs with query string, example: http://localhost:5000/code.html?26482

The query string "?26482" is missing a parameter name, and that's intentional because it's the only way it works.

Adding the parameter causes it to serve without the query string, try: http://localhost:5000/code.html?id=26482 for instance.

When updated to version 7, none of this works (with parameter and without parameter).

Guia do colaborador

Pilha de tecnologia
typescriptnodejs
Domain
backend
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Familiarity with Node.jsUnderstanding of HTTP query strings
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
Investigate how `serve` handles redirects, particularly the logic that constructs redirect URLs. Look in the source code for middleware or route handling that deals with query strings. Check if the issue is related to how the `Content Type` or `Location` headers are set during redirects. Review the comments for any hints about the expected behavior.