ljharb/qs

square brackets confuse the parser

Open

#513 aperta il 10 ott 2024

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)JavaScript (744 fork)batch import
bughelp wantedparsestringify

Metriche repository

Star
 (8015 star)
Metriche merge PR
 (Merge medio 6g 22h) (2 PR mergiate in 30 g)

Descrizione

Hello, while parsing a body like the following, qs fails:

buttons[commands.identifier_orders_merge_action|{"orders":["47441","47440"]}]=Unisci+ordini

This becomes something like:

'buttons[commands.identifier_orders_merge_action|{"orders":': { '"47441","47440"': 'Unisci ordini' }

while it should be something more similar to this:

'buttons[commands.identifier_orders_merge_action|{"orders":{ '"47441","47440"}]': 'Unisci ordini'

Guida contributor

Direzione di ricerca
Indaga la logica di parsing nella libreria qs, probabilmente nel file `lib/parse.js`, per capire come le parentesi quadre e le stringhe JSON annidate vengono tokenizzate. Cerca test esistenti relativi al parsing delle parentesi e considera di aggiungere un caso di test che riproduca il problema con una stringa di query contenente JSON con parentesi. La correzione potrebbe coinvolgere la modifica della regex o della macchina a stati che identifica chiavi e valori. Rivedi eventuali issue o commenti collegati per ulteriore contesto.
Tech stack
javascript
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50