ppkarwasz commented on code in PR #2736:
URL: https://github.com/apache/logging-log4j2/pull/2736#discussion_r1677544025


##########
src/site/antora/modules/ROOT/pages/manual/lookups.adoc:
##########
@@ -16,63 +16,298 @@
 ////
 = Lookups
 
-Lookups provide a way to add values to the Log4j configuration at
-arbitrary places. They are a particular type of Plugin that implements
-the
-link:../javadoc/log4j-core/org/apache/logging/log4j/core/lookup/StrLookup.html[`StrLookup`]
-interface. Information on how to use Lookups in configuration files can
-be found in the xref:manual/configuration.adoc#PropertySubstitution[Property
-Substitution] section of the xref:manual/configuration.adoc[Configuration]
-page.
+Log4j Core provides a flexible and extensible property substitution system 
loosely based on the
+https://commons.apache.org/proper/commons-text/javadocs/api-release/org/apache/commons/text/StringSubstitutor.html[Commons
 Text `StringSubstitutor`].

Review Comment:
   Fixed in fd9f75ed4be43ac9ae53f176d816fca0e0ff1140



##########
src/site/antora/modules/ROOT/pages/manual/lookups.adoc:
##########
@@ -16,63 +16,298 @@
 ////
 = Lookups
 
-Lookups provide a way to add values to the Log4j configuration at
-arbitrary places. They are a particular type of Plugin that implements
-the
-link:../javadoc/log4j-core/org/apache/logging/log4j/core/lookup/StrLookup.html[`StrLookup`]
-interface. Information on how to use Lookups in configuration files can
-be found in the xref:manual/configuration.adoc#PropertySubstitution[Property
-Substitution] section of the xref:manual/configuration.adoc[Configuration]
-page.
+Log4j Core provides a flexible and extensible property substitution system 
loosely based on the
+https://commons.apache.org/proper/commons-text/javadocs/api-release/org/apache/commons/text/StringSubstitutor.html[Commons
 Text `StringSubstitutor`].
+
+The property substitution system is composed of two kinds of elements:
+
+* A string interpolation engine 
(xref:manual/architecture.adoc#StrSubstitutor[`StrSubstitutor`]) that evaluates 
`$+{...}+` expressions.
+See xref:manual/configuration.adoc#property-substitution[property 
substitution] for more details.
+* A set of
+xref:plugin-reference.adoc#org-apache-logging-log4j_log4j-core_org-apache-logging-log4j-core-lookup-StrLookup[`StrLookup`]
+plugins that provide values for simple `$+{prefix:key}+` expressions.

Review Comment:
   Fixed in fd9f75ed4be43ac9ae53f176d816fca0e0ff1140



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