vercel/serve

Make redirections support query strings

Open

Aperta il 30 mag 2018

Vedi su GitHub
 (6 commenti) (5 reazioni) (0 assegnatari)TypeScript (8811 star) (719 fork)batch import
help wantedpatch

Descrizione

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).

Guida contributor

Tech stack
typescriptnodejs
Dominio
backend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Familiarity with Node.jsUnderstanding of HTTP query strings
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65
Direzione di ricerca
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.