ljharb/qs

square brackets confuse the parser

Aperta

#513 aperta il 10 ott 2024

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)JavaScript (911 fork)batch import
bughelp wantedparsestringify

Metriche repository

Star
 (8943 stelle)
Metriche merge PR
 (Metriche PR in attesa)

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