jakartaee / servlet

Jakarta Servlet
https://eclipse.org/ee4j/servlet
Other
262 stars 83 forks source link

Clarify size unit for `MultipartConfigElement` and `MultipartConfig` #463

Closed Marcono1234 closed 1 year ago

Marcono1234 commented 2 years ago

Currently the documentation for both MultipartConfigElement and MultipartConfig mentions the "size" of the uploaded files. Could you please include the unit of that size in the documentation as well, to avoid any confusion. I assume the size is in bytes, so merely replacing "size" with "size in bytes" (or similar) would in my opinion already help.

Also, a bit unrelated but there is a typo in the documentation ("defualts"), might be good to fix it then as well: https://github.com/jakartaee/servlet/blob/5650e7831609ad2b436d7582ef0a39785f317008/api/src/main/java/jakarta/servlet/MultipartConfigElement.java#L37