vy commented on PR #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592#issuecomment-1904116163
@lukaszspyra, I am sorry for the delay. There were more pressing issues. As
a part of our ongoing effort to clean up the `log4j-api` in `2.x`, we are
re-visiting the idea of decreasing
ppkarwasz commented on PR #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592#issuecomment-1903912059
@lukaszspyra,
Sorry for the delayed answer.
Your PR looks good to me, if the default status logger level of `ERROR` can
not be changed.
Personally I don't
lukaszspyra commented on PR #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592#issuecomment-1758014731
> I am not in favor of this. We should fix the original problem that a JSON
or YAML configuration is found but dependencies are missing to log at error
instead.
Looking
rgoers commented on PR #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592#issuecomment-1652657784
I am not in favor of this. We should fix the original problem that a JSON or
YAML configuration is found but dependencies are missing to log at error
instead.
--
This is an
ppkarwasz commented on code in PR #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592#discussion_r1274555963
##
log4j-api/src/main/java/org/apache/logging/log4j/status/StatusLogger.java:
##
@@ -139,7 +139,7 @@ private StatusLogger(
final MessageFactory
lukaszspyra opened a new pull request, #1592:
URL: https://github.com/apache/logging-log4j2/pull/1592
Fix of #1501.
1.
[StatusLogger](https://github.com/apache/logging-log4j2/blob/2.x/log4j-api/src/main/java/org/apache/logging/log4j/status/StatusLogger.java)
default level bumped
