parse: add new option to throw when parameterLimit is reached

[ljharb]

Perhaps instead, add a new option that throws whenever any limit is reached, including arrayLimit.

See expressjs/express#5878

[IamLizu]

Hey @ljharb 👋

I would like to work on this.

[ljharb]

@IamLizu no need to "claim" issues in open source :-) just make a PR for anything that has a "help wanted" label

[IamLizu]

Hey @ljharb 👋

Can we do the implement the arrayLimit check in a different PR? If yes, I would enable #517 for review.

Edit: Yes, we can implement a single option to handle throw for both cases with clear message of which one exceeded. But I was wondering if that could be a re-factor on this.

[ljharb]

I think it'd be important to add it in the same PR, otherwise we'd need two options (to not be a breaking change)

[IamLizu]

(sorry, copying from previous comment)

Yes, we can implement a single option to handle throw for both cases with clear message of which one exceeded. But I was wondering if that could be a re-factor on this.

[ljharb]

If it's released with support for only one limit, then it can't ever support a second limit without a breaking change.

[IamLizu]

Agreed 🤝

And thanks for the early review on the PR 🙏