[New] `parse`: call decoder when parsing objects

ljharb / qs

A querystring parser with nesting support

BSD 3-Clause "New" or "Revised" License

8.47k stars 730 forks source link

[New] `parse`: call decoder when parsing objects #479

Open nbgraham opened 1 year ago

nbgraham commented 1 year ago

Fixes #284.

ljharb commented 1 year ago

This seems like it's either a bugfix - a patch - or a breaking change, in which case it should be implemented behind an option instead.

nbgraham commented 1 year ago

@ljharb Based on this I was treating it as a bugfix. It's kinda already behind the decoder option. If you don't provide that, this won't do anything. But I guess someone could be providing the decoder option today, and this could change the behavior for them.

ljharb commented 1 year ago

That's exactly my concern - especially since this issue was filed five years ago and the behavior probably long predates that.