ramanathan1504 commented on issue #4248:
URL: 
https://github.com/apache/logging-log4j2/issues/4248#issuecomment-5263614087

   @tfmorris Thanks. `elementType` looks like a red herring.                    
                                                                                
                           
                                                                                
                                                                                
                   
   If `IndentingLayout` isn't in the registry its node has a `null` plugin 
type, and `ConsoleAppender`'s inherited `Property[]` element sends the visitor 
down the array branch, which calls `childType.getElementName()` unguarded  
that's the NPE. The single-value branch has had that null check since #1392.    
                                         
                                                                                
                                                                                
                   
   Same NPE with and without `elementType`, and on 2.25.4 and 2.26.0 too, so it 
isn't new in 2.26.1, which version were you on before, and did the JDK change 
too? Your config  also says `packages = com.google.refine.logging` while the 
class is in `com.google.util.logging`, so registration only comes from the 
generated `Log4j2Plugins.dat`  is that in your jar?                             
                                                                                
                                                     
                                                                                
                                                                                
                   
   I'll add the missing null check either way.


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