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.

Sign up for GitHub

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

Jump to bottom

Add config option for QueryStringDecoder semicolonIsNormalChar #11411

Open
yawkat opened this issue Dec 5, 2024 · 0 comments · May be fixed by #11416
Open

Add config option for QueryStringDecoder semicolonIsNormalChar #11411

yawkat opened this issue Dec 5, 2024 · 0 comments · May be fixed by #11416
Labels
info: good first issue Good for newcomers type: improvement A minor improvement to an existing feature
Milestone
4.8.0

Comments

@yawkat
Copy link
Member

yawkat commented Dec 5, 2024

Feature description

Netty's QueryStringDecoder has an option semicolonIsNormalChar that defaults to false. We need a config option that allows changing that setting.

We also have our own copy of QueryStringDecoder in core. It also needs this config option.
@yawkat yawkat added info: good first issue Good for newcomers type: improvement A minor improvement to an existing feature labels Dec 5, 2024
@yawkat yawkat added this to the 4.8.0 milestone Dec 5, 2024
@glorrian glorrian linked a pull request Dec 6, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
info: good first issue Good for newcomers type: improvement A minor improvement to an existing feature
Projects
4.8.0 Release
Status: No status
Milestone
4.8.0
Development

Successfully merging a pull request may close this issue.

Add semicolonIsNormalChar and maxParams properties to the server config glorrian/micronaut-core
1 participant
@yawkat