Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: parsing array of size > 20 in query (#3447) #3448

Merged
merged 1 commit into from Nov 23, 2023

Conversation

ghostoy
Copy link
Contributor

@ghostoy ghostoy commented Nov 21, 2023

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

web

Description of change

通过增加simple模式的解析参数,修复 #3447

例如:在config.default.ts中,设置:

export default {
  web: {
    queryParseMode: 'simple', // 指定使用qs作为解析器
    queryParseOptions: {
      arrayLimit: 100, // 设置将array的解析元素上限为100
    }
  }
  // ... 其它参数
}

@czy88840616 czy88840616 added pr: polish This PR adds a very minor behavior improvement that users will enjoy. pr: bug fix This PR fixes a bug in a past release. and removed pr: polish This PR adds a very minor behavior improvement that users will enjoy. labels Nov 21, 2023
@czy88840616 czy88840616 merged commit 62c0cf7 into midwayjs:main Nov 23, 2023
6 checks passed
@ghostoy ghostoy deleted the feat/simple-parser-options branch November 27, 2023 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: bug fix This PR fixes a bug in a past release.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants