afs commented on code in PR #1645:
URL: https://github.com/apache/jena/pull/1645#discussion_r1036868799
##########
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:
There is no rush to complete this.
@kinow - thanks for catching the issues - I hadn't noticed that the constant
was Jetty-specific.
With various other things doing on, clarifying the situation for Tomcat may
not get done for 4.7.0.
There may be a way to automatically programtically set
`allowCasualMultipartParsing` in Tomcat, otherwise the user needs to set it in
their `context.xml`.
Marked as "draft" to avoid accidentally merging!
--
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]
