You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Fernando Ribeiro (JIRA)" <ji...@apache.org> on 2012/06/26 22:10:43 UTC
[jira] [Created] (FILEUPLOAD-208) CLONE - [fileupload] Use the
ServletRequest encoding as default
Fernando Ribeiro created FILEUPLOAD-208:
-------------------------------------------
Summary: CLONE - [fileupload] Use the ServletRequest encoding as default
Key: FILEUPLOAD-208
URL: https://issues.apache.org/jira/browse/FILEUPLOAD-208
Project: Commons FileUpload
Issue Type: Bug
Affects Versions: 1.0 Final
Environment: Operating System: Windows XP
Platform: PC
Reporter: Fernando Ribeiro
The fileupload package operates on a ServletRequest to conveniently extract it's
content. The ServletRequest.getCharacterEncoding() method "Returns the name of
the character encoding used in the body of this request", and thus should be
used as a default by the fileupload processing as well.
Further, whether from the servlet request or an explicit set on the FileUpload
instance, the encoding should propagate to the FileItem (again by default,
unless it's overridden by a part content type header or explicit set).
Currently, in order to have a UTF-8 request properly parsed, one must first call
request.setCharacterEncoding("UTF-8")
then call
diskFileUpload.setHeaderEncoding("UTF-8")
and finally use
fileItem.getString("UTF-8")
There's no reason for this duplicity - if the first is set, the rest should
propagate by default, unless explicitly overridden. This is a source for bugs
and unneeded extra work (it took quite a while to figure these out, after
examining the source code - this work can be saved for other developers, after
all, that's what this package is for!).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FILEUPLOAD-208) CLONE - [fileupload] Use the
ServletRequest encoding as default
Posted by "Fernando Ribeiro (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/FILEUPLOAD-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13401661#comment-13401661 ]
Fernando Ribeiro commented on FILEUPLOAD-208:
---------------------------------------------
Any change in opinion regarding the "interface breakage" argument?
> CLONE - [fileupload] Use the ServletRequest encoding as default
> ---------------------------------------------------------------
>
> Key: FILEUPLOAD-208
> URL: https://issues.apache.org/jira/browse/FILEUPLOAD-208
> Project: Commons FileUpload
> Issue Type: Bug
> Affects Versions: 1.0 Final
> Environment: Operating System: Windows XP
> Platform: PC
> Reporter: Fernando Ribeiro
>
> The fileupload package operates on a ServletRequest to conveniently extract it's
> content. The ServletRequest.getCharacterEncoding() method "Returns the name of
> the character encoding used in the body of this request", and thus should be
> used as a default by the fileupload processing as well.
> Further, whether from the servlet request or an explicit set on the FileUpload
> instance, the encoding should propagate to the FileItem (again by default,
> unless it's overridden by a part content type header or explicit set).
> Currently, in order to have a UTF-8 request properly parsed, one must first call
> request.setCharacterEncoding("UTF-8")
> then call
> diskFileUpload.setHeaderEncoding("UTF-8")
> and finally use
> fileItem.getString("UTF-8")
> There's no reason for this duplicity - if the first is set, the rest should
> propagate by default, unless explicitly overridden. This is a source for bugs
> and unneeded extra work (it took quite a while to figure these out, after
> examining the source code - this work can be saved for other developers, after
> all, that's what this package is for!).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (FILEUPLOAD-208) CLONE - [fileupload] Use the
ServletRequest encoding as default
Posted by "Fernando Ribeiro (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/FILEUPLOAD-208?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Fernando Ribeiro updated FILEUPLOAD-208:
----------------------------------------
Issue Type: Improvement (was: Bug)
Environment: All supported environments. (was: Operating System: Windows XP
Platform: PC)
Affects Version/s: (was: 1.0 Final)
1.2.2
> CLONE - [fileupload] Use the ServletRequest encoding as default
> ---------------------------------------------------------------
>
> Key: FILEUPLOAD-208
> URL: https://issues.apache.org/jira/browse/FILEUPLOAD-208
> Project: Commons FileUpload
> Issue Type: Improvement
> Affects Versions: 1.2.2
> Environment: All supported environments.
> Reporter: Fernando Ribeiro
>
> The fileupload package operates on a ServletRequest to conveniently extract it's
> content. The ServletRequest.getCharacterEncoding() method "Returns the name of
> the character encoding used in the body of this request", and thus should be
> used as a default by the fileupload processing as well.
> Further, whether from the servlet request or an explicit set on the FileUpload
> instance, the encoding should propagate to the FileItem (again by default,
> unless it's overridden by a part content type header or explicit set).
> Currently, in order to have a UTF-8 request properly parsed, one must first call
> request.setCharacterEncoding("UTF-8")
> then call
> diskFileUpload.setHeaderEncoding("UTF-8")
> and finally use
> fileItem.getString("UTF-8")
> There's no reason for this duplicity - if the first is set, the rest should
> propagate by default, unless explicitly overridden. This is a source for bugs
> and unneeded extra work (it took quite a while to figure these out, after
> examining the source code - this work can be saved for other developers, after
> all, that's what this package is for!).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira