search

ljharb / qs

A querystring parser with nesting support
BSD 3-Clause "New" or "Revised" License
8.55k stars 728 forks source link

[New] `stringify`: throw on cycles, instead of an infinite loop #393

Closed Poyoman39 closed 3 years ago

Poyoman39 commented 3 years ago

Fix issue #367 with "maxDepth" option.

codecov[bot] commented 3 years ago

Codecov Report

Merging #393 (d493679) into master (e40c0e0) will decrease coverage by 0.86%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #393      +/-   ##
==========================================
- Coverage   99.85%   98.98%   -0.87%     
==========================================
  Files           8        8              
  Lines        1363     1378      +15     
  Branches      166      167       +1     
==========================================
+ Hits         1361     1364       +3     
- Misses          2       14      +12
Impacted Files Coverage Δ
lib/stringify.js 100.00% <100.00%> (ø)
test/stringify.js 96.79% <100.00%> (-2.95%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e40c0e0...74dd9d2. Read the comment docs.

Poyoman39 commented 3 years ago

Ok i will provide a Map solution ;)