Details
-
Bug
-
Resolution: Duplicate
-
High
-
None
-
6.2.34
-
None
-
None
Description
Steps to reproduce
- Upload Magnolia to 6.2.30 or more recent
- Create a template with a form that contains 10 fields or more where some fields are file inputs.
- Change form enctype to multipart/form-data
Expected results
Form is sent correctly and you can use the information of the request to do some logic.
Actual results
MultipartRequestFilter throws an exception because it can't handle 10 or more fields, but this should only be for "file" type fields and it is counting also text, checkbox, select, etc... fields.
Workaround
Increase to a big number the value of the property "fileCountMax" for MultipartRequestFilter filter at the configuration in /server/filters/multipartRequest/. For example:
fileCountMax: 100
Development notes
Since in Magnolia 6.2.30 changed to commons-fileupload 1.5.0, this library "allows" to set a max of files you can upload in a request, but this library is not checking only files in the request. Is checking ALL the fields (files or not) in the request.
You can check it in this commit from the github library: https://github.com/apache/commons-fileupload/commit/e20c04990f7420ca917e96a84cec58b13a1b3d17#commitcomment-113912200
Checklists
Attachments
Issue Links
- is duplicated by
-
MAGNOLIA-8889 MultipartRequestFilter#fileCountMax restricts not only files but any param
-
- Closed
-