Apply review suggestions (#2846)

Author: ppkarwasz

log4j-core/src/main/java/org/apache/logging/log4j/core/impl/ThreadContextDataInjector.java

@@ -89,9 +89,9 @@ private static List<ContextDataProvider> getServiceProviders() {
 
     private abstract static class AbstractContextDataInjector implements ContextDataInjector {
 
-        protected final List<ContextDataProvider> providers;
+        final List<ContextDataProvider> providers;
 
-        protected AbstractContextDataInjector() {
+        AbstractContextDataInjector() {
             this.providers = getProviders();
         }
 

src/changelog/.2.x.x/2331_custom_context_data_in_lookup.xml

@@ -2,7 +2,7 @@
 <entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xmlns="https://logging.apache.org/xml/ns"
        xsi:schemaLocation="https://logging.apache.org/xml/ns https://logging.apache.org/xml/ns/log4j-changelog-0.xsd"
-       type="added">
-  <issue id="2331" link="2330_add_faster_web_app_context_map.xml"/>
-  <description format="asciidoc">Add support for custom context data in `ctx` lookup.</description>
+       type="fixed">
+  <issue id="2331" link="https://github.com/apache/logging-log4j2/issues/2331"/>
+  <description format="asciidoc">Fix custom thread-context data provider handling in lookups and filters.</description>
 </entry>