This is an automated email from the ASF dual-hosted git repository.
paulrutter pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/felix-dev.git
commit 6858ded9250434153971e1122202f40c73a4bcf1
Merge: d8bce3a111 ce1b5afef1
Author: Paul <p...@blueconic.com>
AuthorDate: Wed May 7 20:04:25 2025 +0200
Merge pull request #415 from
apache/maintenance/FELIX-6774-IT-demonstrating-the-issue
FELIX-6774 `org.apache.felix.http.jetty.maxFormSize` not enforced: Unit
test to demonstrate issue
- Merging the IT, which now asserts the correct behavior of
`org.apache.felix.http.jetty.maxFormSize`.
context.setMaxFormContentSize(this.config.getMaxFormSize());
Only affects request.getParameter(), but does not apply to any other
places. It does not affect a multipart file upload for example.
For that purpose, one can add the SizeLimitHandler, which affects all
incoming requests (including form data). It just operates on a different level.
In that respect, this is not a bug, but a change request.
...eModeDefaultIT.java => JettyMaxFormSizeIT.java} | 44 +++++++++++++++-------
.../jetty/it/JettyUriComplianceModeDefaultIT.java | 2 +
.../jetty/it/JettyUriComplianceModeLegacyIT.java | 2 +
.../felix/http/jetty/it/JettyVirtualThreadsIT.java | 2 +
4 files changed, 36 insertions(+), 14 deletions(-)
