rombert commented on a change in pull request #24:
URL:
https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/24#discussion_r518827596
##########
File path:
src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverFactoryConfig.java
##########
@@ -149,6 +149,12 @@
" and on the alias update time if the number of aliases
is huge (over 10000).")
boolean resource_resolver_optimize_alias_resolution() default true;
+ @AttributeDefinition(name = "Allowed Optimize alias path",
+ description = "This setting can contain a list of path prefixes, e.g.
/libs/, /content/. If " +
+ "such a list is configured, for alias optimization, only
paths from resources starting with this prefix " +
+ "are considered. If the list is empty, all paths are used.)")
+ String[] resource_resolver_optimize_alias_allowedlist();
Review comment:
I would say that since we started discussing about
https://cwiki.apache.org/confluence/display/SLING/Removal+of+problematic+language
we should not add any more problematic terms like whitelist.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
