lukechilds/node - node - Gitea: Git with a cup of tea

13192 Commits

131 Branches

446 Tags

325 MiB

Tree: 22bb7c9d27

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
Myles Borins	85743c0e92	querystring: check that maxKeys is finite There was a very subtle change in behavior introduced with `27def4f` In the past if querystring.parse was given Infinity for maxKeys, everything worked as expected. Check to see is maxKeys is Infinity before forwarding the value to String.prototype.split which causes this regression PR-URL: https://github.com/nodejs/node/pull/5066 Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>	9 years ago

Myles Borins

85743c0e92

querystring: check that maxKeys is finite

There was a very subtle change in behavior introduced with 27def4f

In the past if querystring.parse was given Infinity for maxKeys,
everything worked as expected.

Check to see is maxKeys is Infinity before forwarding the value to
String.prototype.split which causes this regression

PR-URL: https://github.com/nodejs/node/pull/5066
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>

9 years ago

1 Commits (22bb7c9d27fe495f3d346967a45dc3d226ff7056)