ljharb/qs

square brackets confuse the parser

Open

#513 ouverte le 10 oct. 2024

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)JavaScript (744 forks)batch import
bughelp wantedparsestringify

Métriques du dépôt

Stars
 (8 015 stars)
Métriques de merge PR
 (Merge moyen 6j 22h) (2 PRs mergées en 30 j)

Description

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'

Guide contributeur

Direction de recherche
Étudiez la logique d'analyse dans la bibliothèque qs, probablement dans le fichier `lib/parse.js`, pour comprendre comment les crochets et les chaînes JSON imbriquées sont tokenisés. Recherchez les tests existants liés à l'analyse des crochets et envisagez d'ajouter un cas de test qui reproduit le problème avec une chaîne de requête contenant du JSON avec des crochets. La correction peut impliquer d'ajuster l'expression régulière ou la machine à états qui identifie les clés et les valeurs. Consultez les problèmes ou commentaires liés pour obtenir un contexte supplémentaire.
Stack technique
javascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50