vy commented on PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#issuecomment-3209253360
Another concern I have regarding adding a new, as a matter of fact, 2nd
after `getSupportedTypes()`, _"supported file extensions"_ concept to `CF` is
that it reinforces the idea of a `C
vy commented on PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#issuecomment-3209220815
> > 1. It duplicates quite some logic (e.g., in `JsonConfigurationFactory`,
extensions and suffixes are semantically identical).
>
> Totally agree. Let’s put a **default** impleme
yybmion commented on code in PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#discussion_r2289835030
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/ConfigurationFactory.java:
##
@@ -617,4 +636,19 @@ private String[] parseConfigLocations(final S
github-actions[bot] commented on issue #2373:
URL:
https://github.com/apache/logging-log4j2/issues/2373#issuecomment-3208720064
This issue is stale because it has been waiting for your feedback for more
than 60 days. The Apache Logging Services community values every submitted
issue, but w
github-actions[bot] closed issue #3621: Used on Routes, CronTriggeringPolicy is
not executed correctly
URL: https://github.com/apache/logging-log4j2/issues/3621
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
github-actions[bot] closed issue #3622: Failover Appender logic is not working
properly
URL: https://github.com/apache/logging-log4j2/issues/3622
--
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
github-actions[bot] closed issue #3726: log messages with last parameter being
a Throwable log a warning
URL: https://github.com/apache/logging-log4j2/issues/3726
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
rm5248 commented on code in PR #524:
URL: https://github.com/apache/logging-log4cxx/pull/524#discussion_r2289604752
##
src/site/markdown/configuration-samples.md:
##
@@ -40,12 +40,57 @@ of [LoggerRepository](@ref log4cxx.spi.LoggerRepository).
To use automatic configuration w
swebb2066 commented on PR #524:
URL: https://github.com/apache/logging-log4cxx/pull/524#issuecomment-3208554631
The [staging site](https://logging.staged.apache.org/log4cxx/1.6.0) is
available for review
--
This is an automated message from the Apache Git Service.
To respond to the messag
swebb2066 merged PR #523:
URL: https://github.com/apache/logging-log4cxx/pull/523
--
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: notifications-unsubscr
swebb2066 commented on code in PR #524:
URL: https://github.com/apache/logging-log4cxx/pull/524#discussion_r2289506789
##
src/site/markdown/configuration-samples.md:
##
@@ -40,12 +40,57 @@ of [LoggerRepository](@ref log4cxx.spi.LoggerRepository).
To use automatic configuratio
ramanathan1504 commented on issue #3804:
URL:
https://github.com/apache/logging-log4j2/issues/3804#issuecomment-3207740696
Hi @ppkarwasz ,
Thanks a lot for the detailed context and for linking the reproducibility
discussion.
Here’s my plan:
1. Reproduce consistent
ramanathan1504 commented on code in PR #3888:
URL: https://github.com/apache/logging-log4j2/pull/3888#discussion_r2288997691
##
log4j-core-test/src/test/java/org/apache/logging/log4j/core/pattern/NamedInstantPatternTest.java:
##
@@ -40,6 +41,19 @@ void compatibilityOfLegacyPatte
ppkarwasz commented on PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#issuecomment-3206986816
> 1. It duplicates quite some logic (e.g., in `JsonConfigurationFactory`,
extensions and suffixes are semantically identical).
Totally agree. Let’s put a **default** implem
ppkarwasz commented on code in PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#discussion_r2288444366
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/ConfigurationFactory.java:
##
@@ -263,6 +264,14 @@ public static void removeConfigurationFacto
vy commented on code in PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#discussion_r2288091615
##
log4j-core/src/main/java/org/apache/logging/log4j/core/config/ConfigurationFactory.java:
##
@@ -617,4 +636,19 @@ private String[] parseConfigLocations(final String
vy commented on PR #3839:
URL: https://github.com/apache/logging-log4j2/pull/3839#issuecomment-3206278760
@ppkarwasz, would you mind reminding me why we decided to introduce a new
API such that
1. It duplicates quite some logic (e.g., in `JsonConfigurationFactory`,
extensions and suf
rm5248 commented on code in PR #524:
URL: https://github.com/apache/logging-log4cxx/pull/524#discussion_r2287818198
##
src/site/markdown/configuration-samples.md:
##
@@ -40,12 +40,57 @@ of [LoggerRepository](@ref log4cxx.spi.LoggerRepository).
To use automatic configuration w
18 matches
Mail list logo