[ https://issues.apache.org/jira/browse/FILEUPLOAD-137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13598673#comment-13598673 ]
Simone Tripodi commented on FILEUPLOAD-137: ------------------------------------------- while it would be possible to work around the NPE on {{boundary}}, the adapted {{InputStream}} reference cannot be set if not via the constructor since {{MultipartStream#input}} is {{final}}. Since it doesn't look like there is a fix for this, I'd go for {{UnsupportedOperationException}} in {{MultipartStream()}} constructor and add a note in its javadoc. Thoughts? > MultipartStream public API broken > --------------------------------- > > Key: FILEUPLOAD-137 > URL: https://issues.apache.org/jira/browse/FILEUPLOAD-137 > Project: Commons FileUpload > Issue Type: Bug > Affects Versions: 1.2 > Reporter: Mark Sinke > Assignee: Jochen Wiedmann > Fix For: 1.2.1 > > > In commons-transaction 1.2 the MultipartStream class has 2 public > constructors. Both are deprecated; however their implementation delegates to > non-visible (package-private) constructors. There are two issues here: > 1. the deprecated, delegating constructors use a null pointer for the > progress notifier, which in turn yield a NullPointerException when you try to > use them > 2. the non-deprecated constructors are not visible. > Hence, I cannot really upgrade from 1.0 to 1.2. > Thanks, > Mark. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira