fix(deps): update version lib qs by gabrieel1007 · Pull Request #6969 · expressjs/express (original) (raw)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @gabrieel1007! Thanks for contributing to Express.

We likely do not need to add additional tests here. This behavior is already well covered in qs (see: ljharb/qs@3086902), and Express does not define an arrayLimit by default.

As a result, customization is left to the user. Any related testing should ideally be handled at the application level, according to their specific configuration and requirements.

Additionally, we will probably want to open a separate PR targeting the 4.x branch to upgrade the qs version as well, since this PR is intended to apply only to the Express 5 line.