Re: [I] MdcPatternConverter - update output format for log process tool friendly and smaller size (logging-log4j2)

2025-02-06 Thread via GitHub
vy commented on issue #3435: URL: https://github.com/apache/logging-log4j2/issues/3435#issuecomment-2639652617 > When a log string contains `key=value` format, in modern log search tool, like [Splunk](https://www.splunk.com/) or [ELK](https://www.elastic.co/): @amosshi, it sounds cou

Re: [PR] Publish build scans to develocity.apache.org [logging-parent]

2025-02-06 Thread via GitHub
vy commented on PR #313: URL: https://github.com/apache/logging-parent/pull/313#issuecomment-2639113171 @ppkarwasz, will do. @gmcdonald, thanks so much for your kind help and patience. I will try to cut a release either today or sometime Saturday. I will also make sure to ping you, s

Re: [I] Provide path of expected .config file in error message (logging-log4net)

2025-02-06 Thread via GitHub
FreeAndNil closed issue #227: Provide path of expected .config file in error message URL: https://github.com/apache/logging-log4net/issues/227 -- 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 spe

Re: [PR] Show config file path in error message (logging-log4net)

2025-02-06 Thread via GitHub
FreeAndNil merged PR #228: URL: https://github.com/apache/logging-log4net/pull/228 -- 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-unsubsc

Re: [I] Provide path of expected .config file in error message (logging-log4net)

2025-02-06 Thread via GitHub
FreeAndNil closed issue #227: Provide path of expected .config file in error message URL: https://github.com/apache/logging-log4net/issues/227 -- 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 spe

[I] Provide path of application's .config file in error message (logging-log4net)

2025-02-06 Thread via GitHub
FrankNuessle opened a new issue, #227: URL: https://github.com/apache/logging-log4net/issues/227 I got the message: "log4net:ERROR Failed to find configuration section 'log4net' in the **application's .config file**. Check your **.config** file for the and elements. The configuration s

Re: [I] Compiling on Module Path with -Werror and -Xlint:all (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on issue #3437: URL: https://github.com/apache/logging-log4j2/issues/3437#issuecomment-2640572981 > This would be in line with the `requires static org.jspecify` you already have and it should do no harm. If you agree and if you give me a pointer how this would be added

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
Suvrat1629 commented on PR #3394: URL: https://github.com/apache/logging-log4j2/pull/3394#issuecomment-2640140229 I am closing this pr, will open a new one when i am done with the issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
Suvrat1629 closed pull request #3394: Bug Fix: Script Resolution fix URL: https://github.com/apache/logging-log4j2/pull/3394 -- 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

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3394: URL: https://github.com/apache/logging-log4j2/pull/3394#issuecomment-2640620357 > Could you verify the log4j-script-ref-test.xml? Also I did try writing the test cases my self and I was getting the ScriptManager == null error everytime. How do i handle this?

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3394: URL: https://github.com/apache/logging-log4j2/pull/3394#issuecomment-2640632320 > Could you verify the log4j-script-ref-test.xml? The `log4j-script-ref-test.xml` does not contain any Log4j plugin that uses scripts (see [the list in my comment above](h

Re: [PR] Improve configuration error handling of HttpAppender (logging-log4j2)

2025-02-06 Thread via GitHub
github-actions[bot] commented on PR #3438: URL: https://github.com/apache/logging-log4j2/pull/3438#issuecomment-2640635798 Job Requested goals Build Tool Version Build Outcome Build ScanĀ® build-

Re: [PR] Improve configuration error handling of HttpAppender (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3438: URL: https://github.com/apache/logging-log4j2/pull/3438#issuecomment-2640956633 There are formatting errors in your PR. Run: ``` ./mvnw spotless:apply ``` to fix them. -- This is an automated message from the Apache Git Service. To respo

Re: [PR] Minimize lock usage in `InternalLoggerRegistry` (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3418: URL: https://github.com/apache/logging-log4j2/pull/3418#issuecomment-2640302141 Thanks, I added it to the backlog #3161 -- 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

[PR] Improve configuration error handling of HttpAppender (logging-log4j2)

2025-02-06 Thread via GitHub
Suvrat1629 opened a new pull request, #3438: URL: https://github.com/apache/logging-log4j2/pull/3438 This PR introduces improvements to HttpAppender and adds a new test class, HttpAppenderBuilderTest, to enhance test coverage. The changes include: Updating HttpAppender.java to improve log

Re: [PR] Prevent timeout fault running JSONLayoutFuzzer [logging-log4cxx]

2025-02-06 Thread via GitHub
swebb2066 merged PR #478: URL: https://github.com/apache/logging-log4cxx/pull/478 -- 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: [I] Provide path of application's .config file in error message (logging-log4net)

2025-02-06 Thread via GitHub
FreeAndNil commented on issue #227: URL: https://github.com/apache/logging-log4net/issues/227#issuecomment-2640157697 @FrankNuessle that's a good idea. I will adjust the error message. You can set the assembly for which log4net searches a config file via ```csharp log4net.

[I] Compiling on Module Path with -Werror and -Xlint:all (logging-log4j2)

2025-02-06 Thread via GitHub
jjohannes opened a new issue, #3437: URL: https://github.com/apache/logging-log4j2/issues/3437 This is about getting an error like the following when compiling with a `module-info.java` and the compiler flags `-Xlint:all` and `-Werror`: ``` > Compilation failed; see the compiler ou

[PR] Show config file path in error message (logging-log4net)

2025-02-06 Thread via GitHub
FreeAndNil opened a new pull request, #228: URL: https://github.com/apache/logging-log4net/pull/228 fixes #227 -- 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 unsubscrib

Re: [PR] Minimize lock usage in `InternalLoggerRegistry` (logging-log4j2)

2025-02-06 Thread via GitHub
vy commented on PR #3418: URL: https://github.com/apache/logging-log4j2/pull/3418#issuecomment-2640286754 @ppkarwasz, ported to `2.x`. `main` needs _some_ work, since #3199 needs to be ported too. -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [I] Compiling on Module Path with -Werror and -Xlint:all (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on issue #3437: URL: https://github.com/apache/logging-log4j2/issues/3437#issuecomment-2640456549 @jjohannes, > (1) This was already discussed [here](https://github.com/apache/logging-log4j2/issues/3110#issuecomment-2423586754). The dependencies marked as `` [he

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
Suvrat1629 commented on PR #3394: URL: https://github.com/apache/logging-log4j2/pull/3394#issuecomment-2640475379 Could you verify the log4j-script-ref-test.xml? Also I did try writing the test cases my self and I was getting the ScriptManager == null error everytime. How do i handle this

Re: [I] Compiling on Module Path with -Werror and -Xlint:all (logging-log4j2)

2025-02-06 Thread via GitHub
jjohannes commented on issue #3437: URL: https://github.com/apache/logging-log4j2/issues/3437#issuecomment-2640495652 Thanks for the quick response @ppkarwasz. I'll prepare a PR for the Gradle Metadata in the next days. > A `requires static transitive` statement has some unwant

Re: [PR] Minimize lock usage in `InternalLoggerRegistry` (logging-log4j2)

2025-02-06 Thread via GitHub
vy merged PR #3418: URL: https://github.com/apache/logging-log4j2/pull/3418 -- 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...@lo

Re: [I] Suspect deadlock in InternalLoggerRegistry (intermittent) (logging-log4j2)

2025-02-06 Thread via GitHub
vy closed issue #3399: Suspect deadlock in InternalLoggerRegistry (intermittent) URL: https://github.com/apache/logging-log4j2/issues/3399 -- 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

Re: [I] Suspect deadlock in InternalLoggerRegistry (intermittent) (logging-log4j2)

2025-02-06 Thread via GitHub
vy commented on issue #3399: URL: https://github.com/apache/logging-log4j2/issues/3399#issuecomment-2639959104 Fixed by #3418. -- 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 commen

Re: [PR] Bug Fix: Script Resolution fix (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3394: URL: https://github.com/apache/logging-log4j2/pull/3394#issuecomment-2640358227 @Suvrat1629, I would prefer if you keep this PR open. Due to FOSDEM, I didn't have time to look at this PR, but it is going in the right direction. -- This is an automat

Re: [PR] Minimize lock usage in `InternalLoggerRegistry` (logging-log4j2)

2025-02-06 Thread via GitHub
ppkarwasz commented on PR #3418: URL: https://github.com/apache/logging-log4j2/pull/3418#issuecomment-2640091554 @vy, Thanks for merging this. Could you cherry-pick it to `2.x` and `main` too? :pleading_face: -- This is an automated message from the Apache Git Service. To respond