ljharb/qs

Object with array containing object and scalars is not parsed correctly.

Open

#197 opened on 2017年2月9日

GitHub で見る
 (7 comments) (0 reactions) (1 assignee)JavaScript (8,015 stars) (744 forks)batch import
bughelp wantedparse

説明

Client request from any browser $.get('/ajax', {arr: [{foo: 123, bar: 'foo'},2,3] }, ...

On server (node .10, 4.x and 7.x) with latest qs

is parsed as

{ arr: { __0: [{ foo: 123, bar: 'foo', 2: true}, 3] } }

it should be

{ arr: [ {foo: 123, bar: 'foo'}, 2, 3 ] }

コントリビューターガイド

技術スタック
javascriptnodejs
領域
backendapi
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
JavaScriptunderstanding of query string parsing
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
Look at the qs source code, specifically the parse function in lib/parse.js. The issue involves array parsing with mixed object and scalar values. The current behavior creates an object with numeric keys instead of an array. Check if there is a test file for this case (e.g., test/parse.js). The discussion in the issue suggests that the fix may involve handling array indices correctly when the array contains both objects and primitives. Review any linked PRs or comments for potential solutions.
Object with array containing object and scalars is not parsed correctly. · ljharb/qs#197 | Good First Issue