cziegeler commented on code in PR #75:
URL:
https://github.com/apache/sling-org-apache-sling-engine/pull/75#discussion_r3272934822
##########
src/main/java/org/apache/sling/engine/impl/SlingMainServlet.java:
##########
@@ -90,6 +91,10 @@ public class SlingMainServlet extends GenericServlet {
@Reference
private volatile SlingRequestProcessorImpl requestProcessorImpl;
+ /** extra config properties for multipart file upload support */
+ @Reference
+ private transient RequestParameterConfig reqParamConfig;
Review Comment:
I personally think that this we should ignore such default sonar reports.
While this makes sense in general for Serializable objects, it does imho not
make sense for Servlets (or any other service). These objects will never be
serialized and therefore this just clutters the code
--
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]
