kwin commented on code in PR #10:
URL: 
https://github.com/apache/sling-org-apache-sling-servlets-get/pull/10#discussion_r1096169980


##########
src/main/java/org/apache/sling/servlets/get/impl/DefaultGetServlet.java:
##########
@@ -107,19 +108,19 @@ public class DefaultGetServlet extends 
SlingSafeMethodsServlet {
         String[] index_files() default { "index","index.html" };
 
         @AttributeDefinition(name = "Enable HTML",
-                description = "Whether the renderer for HTML of the default 
GET servlet is enabled or not. By default the HTML renderer is enabled.")
+                description = "Whether the renderer for HTML of the default 
GET servlet is enabled for extension \"html\" or not. By default the HTML 
renderer is enabled.")
         boolean enable_html() default true;

Review Comment:
   Probably we should remove the sentence 
   
   > By default the HTML renderer is enabled.
   
   as the default handling does not really matter (and is to a certain degree 
visually exposed in the Felix Web Console anyhow)



-- 
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]

Reply via email to