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

Review FormFields.MAX_FORM_[KEYS|CONTENT] for core #11385

Closed
sbordet opened this issue Feb 7, 2024 · 0 comments · Fixed by #11386
Closed

Review FormFields.MAX_FORM_[KEYS|CONTENT] for core #11385

sbordet opened this issue Feb 7, 2024 · 0 comments · Fixed by #11386
Labels
Bug For general bugs on Jetty side

Comments

@sbordet
Copy link
Contributor

sbordet commented Feb 7, 2024 •
edited
Loading

Jetty version(s)
12+

Description
FormFields.MAX_FORM_[KEYS|LENGTH] should be reviewed for core, as the behavior is different from eeN.

Core should have good defaults like eeN does.

ContextHandler should have a setter like ServletContextHandler does for embedded usage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug For general bugs on Jetty side
Projects
No open projects
Jetty 12.0.7 - FROZEN
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Making FormFields get defaults from Context, not Request jetty/jetty.project
1 participant
@sbordet