ramanathan1504 commented on PR #4414:
URL: https://github.com/apache/logging-log4j2/pull/4414#issuecomment-6301246665

   The new test fails on `2.x` and passes with the fix, and both branches of 
the new check are covered.
   With this change `logger="com.foo."` and `logger=""` no longer match 
`com.foo.Bar`, and both did before. Can we keep those two working and test 
them? The `createPolicy` javadoc in `LoggerNameLevelRewritePolicy.java` ("all 
event logger names that start with this string") and "logger name prefix" in 
`delegating.adoc` also need to match the new behaviour.


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