panbingkun commented on issue #2337:
URL: 
https://github.com/apache/logging-log4j2/issues/2337#issuecomment-1974257636

   > > Sure, let we to do it now.
   > 
   > @panbingkun, the CI job has successfully completed – needed a kick. Would 
you mind giving `2.23.1-SNAPSHOT` a try and sharing the outcome, please?
   
   @vy @ppkarwasz 
   
   After testing, it was found that it still did `not` output the expected log 
information.
   I analyzed it and found that `StatusLogger.isEnable` is working properly 
now, but the method `StatusConsoleListener.log` still seems to have `some 
issues`, as follows:
   
https://github.com/apache/logging-log4j2/blob/c5420410df2741a7704a0efc8b77b25aca1ddea8/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java#L135-L141
   <img width="610" alt="image" 
src="https://github.com/apache/logging-log4j2/assets/15246973/00c9148a-ba4b-4358-873e-1c7959506f88";>
   
   The `above method` does not consider whether the `debugging mode` is turned 
on,
   So, I proposed a follow-up pr: 
https://github.com/apache/logging-log4j2/pull/2340,
   Because I only have a `rough understanding` of `log4j2`, if the above 
modifications are incorrect, please correct me. Thank you very much!
   


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