👍 I'll prepare a pr.

On Sun., Mar. 13, 2022, 1:11 p.m. Mark Thomas, <ma...@apache.org> wrote:

> On 13/03/2022 03:07, Raymond Augé wrote:
> > tomcat-catalina imports org.apache.tomcat.util.http.fileupload.impl [1]
> > tomcat-coyote however does not export
> > org.apache.tomcat.util.http.fileupload.impl
> >
> > How would you like to address this?
> >
> > Should tomcat-coyote simply export the package?
>
> Seems like the obvious choice.
>
> > or should the classes (two
> > Exceptions) be made part of the Coyote API (not a breaking change to
> > outsiders since it's like adding never before seen types.)
>
> Do you mean move those classes to a new package? If so, exporting looks
> better as these classes are a packaged renamed form of Commons File
> Upload so keeping classes in the existing is preferred.
>
> Mark
>
>
> > This affects all versions from what I can see. It's not a critical issue,
> > just a packaging and modularity one. It WOULD however come into play in
> > OSGi or JPMS use cases.
> >
> > Thoughts?
> >
> > Ray
> >
> > [1]
> >
> https://github.com/apache/tomcat/blob/main/java/org/apache/catalina/connector/Request.java#L110-L111
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>

Reply via email to