Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [I] Initialization deadlock on JaveEE server startup [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [I] Used on Routes, CronTriggeringPolicy is not executed correctly [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [I] Failover Appender logic is not working properly [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [I] log messages with last parameter being a Throwable log a warning [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add a properties file example to configuration documentation [logging-log4cxx]

2025-08-20 Thread via GitHub
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

Re: [PR] Add a properties file example to configuration documentation [logging-log4cxx]

2025-08-20 Thread via GitHub
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

Re: [PR] Improve the 'Logging With Multiple Processes' web-site information [logging-log4cxx]

2025-08-20 Thread via GitHub
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

Re: [PR] Add a properties file example to configuration documentation [logging-log4cxx]

2025-08-20 Thread via GitHub
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

Re: [I] Investigate SBOM Irreproducibility in `log4j-bom` [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] [LOG4J2-3805] NamedInstantPatternTest#compatibilityOfLegacyPattern fails in timezones with minute offsets (e.g., GMT+05:30) [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add API to list supported configuration file extensions [logging-log4j2]

2025-08-20 Thread via GitHub
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

Re: [PR] Add a properties file example to configuration documentation [logging-log4cxx]

2025-08-20 Thread via GitHub
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