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

Making FormFields get defaults from Context, not Request #11386

Merged

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Feb 7, 2024

Instead of taking defaults from Request attributes, take them from the Context attributes.

Fixes: #11385

@joakime joakime added this to the 12.0.x milestone Feb 7, 2024
@joakime joakime requested review from gregw and sbordet February 7, 2024 18:00
@joakime joakime self-assigned this Feb 7, 2024
@joakime joakime linked an issue Feb 7, 2024 that may be closed by this pull request
@joakime joakime merged commit e5d95b4 into jetty-12.0.x Feb 8, 2024
8 checks passed
@joakime joakime deleted the fix/12.0.x/formfields-default-from-context-not-request branch February 28, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Review FormFields.MAX_FORM_[KEYS|CONTENT] for core
2 participants