afs commented on code in PR #1645:
URL: https://github.com/apache/jena/pull/1645#discussion_r1036492716
##########
jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/system/DataUploader.java:
##########
@@ -102,6 +102,10 @@ public static UploadDetails incomingData(HttpAction
action, StreamRDF dest) {
}
}
+ // Jetty requires a setting of this annotation object as a request
attribute.
+ private static MultipartConfigElement multipartConfigElement = new
MultipartConfigElement("");
+ private static String multipartAttributeName =
org.eclipse.jetty.server.Request.__MULTIPART_CONFIG_ELEMENT;
Review Comment:
> Do you know if this __MULTIPART_CONFIG_ELEMENT attribute was created this
way to discourage external use?
I'll check some more. Using the annotation `@MultipartConfig` on
FusekiFilter does not work (it's not a servlet).
The name is Jetty specific which is why it is protected by the
`getAttribute`. Jetty source code likes leading `_`'s. See
`org.eclipse.jetty.server.Request`.
I'm unsure if Tomcat needs anything special. I haven't managed to find a
mention one way or the other.
PS:
https://stackoverflow.com/questions/8047173/how-to-use-httpservletrequestgetparts-in-a-servlet-filter-running-on-tomcat
got me to:
https://tomcat.apache.org/tomcat-7.0-doc/config/context.html#Common_Attributes
Set `allowCasualMultipartParsing`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
