from mobile (sorry for typos ;)
On Tue, Jul 11, 2023, 18:58 Romain Manni-Bucau <rmannibu...@gmail.com> wrote: > @Maxim: exactly, this is why hosting it only in tomcat and using the > servlet abstraction is saner since commons is not usable in tomcat without > relocation or hacks nobdoy wants on any sides and it is the main relevant > consumer these days. > Will the same API be available in Jetty for ex.? IMO code need to stay in Commons > Romain Manni-Bucau > @rmannibucau <https://twitter.com/rmannibucau> | Blog > <https://rmannibucau.metawerx.net/> | Old Blog > <http://rmannibucau.wordpress.com> | Github < > https://github.com/rmannibucau> | > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book > < > https://www.packtpub.com/application-development/java-ee-8-high-performance > > > > > Le mar. 11 juil. 2023 à 13:47, Maxim Solodovnik <solomax...@gmail.com> a > écrit : > > > Tomcat uses the code copy\pasted from commons-fileupload (due to there > > was not release) > > IMO it is better to use library instead of copy/pasting here and there > > ..... > > > > On Tue, 11 Jul 2023 at 18:03, Romain Manni-Bucau <rmannibu...@gmail.com> > > wrote: > > > > > > This is a good point but guess it is not hard - and would be great - to > > fix > > > it in tomcat, just needs to not use ArrayList but a lazy list which > will > > > behave like fileupload streaming API. Under the hood the API already > uses > > > the FileItemIterator , it just needs to be able to populate the list > > lazily. > > > > > > Romain Manni-Bucau > > > @rmannibucau <https://twitter.com/rmannibucau> | Blog > > > <https://rmannibucau.metawerx.net/> | Old Blog > > > <http://rmannibucau.wordpress.com> | Github < > > https://github.com/rmannibucau> | > > > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book > > > < > > > https://www.packtpub.com/application-development/java-ee-8-high-performance > > > > > > > > > > > > Le mar. 11 juil. 2023 à 12:57, Eugene Grybinnyk > > > <euge...@backbase.com.invalid> a écrit : > > > > > > > We tried to get rid of commons-fileupload in favour of Servlet API > for > > our > > > > project where we disable multipart and use streaming (not to store > > files on > > > > the disk). And it looks like Servlet API doesn’t provide a convenient > > way > > > > of streaming the attachments (like commons-fileupload does) so in the > > end > > > > we concluded that we need to implement something similar to what we > get > > > > from commons-fileupload OR to wait for version > commons-fileupload:2.0. > > > > > > > > On 2023/07/09 14:45:21 Emmanuel Bourg wrote: > > > > > Dumb question: what's the benefit of using commons-fileupload in > 2023 > > > > > instead of the equivalent file upload feature of the Servlet API? > > > > > > > > > > The use case I had in mind was to support file upload in very old > > > > > Servlet containers still in production (Tomcat 6 or Jetty 7 for > > example, > > > > > both EOL), but servers supporting the Jakarta API are recent and > have > > > > > the file upload feature integrated. I'd expect commons-fileupload > to > > go > > > > > to dormant in the near future rather than adapted for the jakarta > > > > namespace. > > > > > > > > > > What did I miss? > > > > > > > > > > Emmanuel Bourg > > > > > > > > > > > --------------------------------------------------------------------- > > > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > > > > > > > > > > > > -- > > Best regards, > > Maxim > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > >
