Re: [PR] Bump commons-logging:commons-logging from 1.3.2 to 1.3.3 (logging-log4j2)

2024-07-05 Thread via GitHub
github-actions[bot] closed pull request #2714: Bump commons-logging:commons-logging from 1.3.2 to 1.3.3 URL: https://github.com/apache/logging-log4j2/pull/2714 -- 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

Re: [PR] Bump co.elastic.clients:elasticsearch-java from 8.14.1 to 8.14.2 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2711: URL: https://github.com/apache/logging-log4j2/pull/2711#issuecomment-2211680179 @dependabot rebase -- 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 co

Re: [PR] Bump commons-logging:commons-logging from 1.3.2 to 1.3.3 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2714: URL: https://github.com/apache/logging-log4j2/pull/2714#issuecomment-2211680113 @dependabot rebase -- 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 co

Re: [PR] Bump net.javacrumbs.json-unit:json-unit from 2.39.0 to 2.40.0 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2708: URL: https://github.com/apache/logging-log4j2/pull/2708#issuecomment-2211679853 @dependabot rebase -- 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 co

Re: [I] `StrSubstitutor` cannot parse nested braces (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on issue #2679: URL: https://github.com/apache/logging-log4j2/issues/2679#issuecomment-2211679687 Looking at [bash parameter expansion](https://www.gnu.org/software/bash/manual/html_node/Shell-Parameter-Expansion.html), I would propose to just introduce a simple escape

Re: [PR] Bump flapdoodle-embed.version from 4.11.0 to 4.12.0 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2707: URL: https://github.com/apache/logging-log4j2/pull/2707#issuecomment-2211679784 @dependabot rebase -- 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 co

Re: [PR] Bump net.javacrumbs.json-unit:json-unit from 2.38.0 to 2.40.0 (logging-log4j2)

2024-07-05 Thread via GitHub
github-actions[bot] closed pull request #2705: Bump net.javacrumbs.json-unit:json-unit from 2.38.0 to 2.40.0 URL: https://github.com/apache/logging-log4j2/pull/2705 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Bump org.apache.groovy:groovy-bom from 4.0.21 to 4.0.22 (logging-log4j2)

2024-07-05 Thread via GitHub
github-actions[bot] closed pull request #2699: Bump org.apache.groovy:groovy-bom from 4.0.21 to 4.0.22 URL: https://github.com/apache/logging-log4j2/pull/2699 -- 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

Re: [PR] Bump net.javacrumbs.json-unit:json-unit from 2.38.0 to 2.40.0 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2705: URL: https://github.com/apache/logging-log4j2/pull/2705#issuecomment-2211655535 @dependabot rebase -- 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 co

Re: [PR] Bump org.apache.groovy:groovy-bom from 4.0.21 to 4.0.22 (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2699: URL: https://github.com/apache/logging-log4j2/pull/2699#issuecomment-2211654253 @dependabot rebase -- 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 co

Re: [PR] Revamp `cloud.adoc` for service-oriented architectures (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on code in PR #2715: URL: https://github.com/apache/logging-log4j2/pull/2715#discussion_r1667084479 ## src/site/antora/modules/ROOT/pages/manual/cloud.adoc: ## @@ -15,549 +15,267 @@ limitations under the License. -= Using Log4j in Cloud Enabled Appl

Re: [PR] Consolidate stack trace rendering in Pattern Layout (logging-log4j2)

2024-07-05 Thread via GitHub
alan0428a commented on PR #2691: URL: https://github.com/apache/logging-log4j2/pull/2691#issuecomment-2211071869 @vy Just want to make sure we are on the same page. So for `xxxThrowablePatternConverter#format()`, the existing implementation is that when `event.getThrownProxy()` i

[PR] Revamp the `Filters` page (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz opened a new pull request, #2717: URL: https://github.com/apache/logging-log4j2/pull/2717 In this PR we: - add a section about the filtering process, that explains in detail all the places, where a filter can be configured. - add the description of 3 filters, - improve

[PR] Revamp the `Extending`, `Plugins`, `Architecture`, and `Programmatic configuration` pages (logging-log4j2)

2024-07-05 Thread via GitHub
vy opened a new pull request, #2716: URL: https://github.com/apache/logging-log4j2/pull/2716 This *incomplete* work ports #2696 to `main`. The following tasks still need to be done: - [ ] Adapt `plugins.adoc` for Log4j 3 - Artifact has changed - Package paths have changed

Re: [PR] StrSubstitutor cannot parse nested braces (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on PR #2712: URL: https://github.com/apache/logging-log4j2/pull/2712#issuecomment-2210973106 @ngocnhan-tran1996, Thank you for the PR. Unfortunately this does not solve #2679, but just your particular problem with the `%d{...}` pattern converter as default value

Re: [I] `StrSubstitutor` cannot parse nested braces (logging-log4j2)

2024-07-05 Thread via GitHub
ngocnhan-tran1996 commented on issue #2679: URL: https://github.com/apache/logging-log4j2/issues/2679#issuecomment-2210918728 Hi @vy , @ppkarwasz I have a PR but I think this has more edge case. Please take a look and tell me if I miss something https://github.com/apache/logg

[PR] Revamp `cloud.adoc` for service-oriented architectures (logging-log4j2)

2024-07-05 Thread via GitHub
vy opened a new pull request, #2715: URL: https://github.com/apache/logging-log4j2/pull/2715 Closes #2510. -- 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

[jira] [Work logged] (LOG4J2-3627) PatternLayout: %xEx{ [ "short" | depth]} not working

2024-07-05 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3627?focusedWorklogId=924623&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-924623 ] ASF GitHub Bot logged work on LOG4J2-3627: -- Author: ASF GitHub Bot

Re: [PR] fix: LOG4J2-3627 and gh issue 1729 (logging-log4j2)

2024-07-05 Thread via GitHub
vy commented on PR #2691: URL: https://github.com/apache/logging-log4j2/pull/2691#issuecomment-2210702563 > I saw at `xxxThrowablePatternConverter#format()` we do conditions checking whether `event.getThrownProxy() is null`. But when will it be null? I presume, since `ThrowableProxy`

Re: [PR] Bump commons-logging:commons-logging from 1.3.2 to 1.3.3 (logging-log4j2)

2024-07-05 Thread via GitHub
github-actions[bot] closed pull request #2713: Bump commons-logging:commons-logging from 1.3.2 to 1.3.3 URL: https://github.com/apache/logging-log4j2/pull/2713 -- 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] Revamp the explanation on creating loggers [logging-log4j-kotlin]

2024-07-05 Thread via GitHub
vy opened a new pull request, #79: URL: https://github.com/apache/logging-log4j-kotlin/pull/79 Tipped by @breun, users find it confusing which logger creation mechanism to choose from. This PR is improves the documentation in that direction. -- This is an automated message from the Apache

Re: [PR] Bump dependabot/fetch-metadata from 2.1.0 to 2.2.0 [logging-parent]

2024-07-05 Thread via GitHub
github-actions[bot] closed pull request #200: Bump dependabot/fetch-metadata from 2.1.0 to 2.2.0 URL: https://github.com/apache/logging-parent/pull/200 -- 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 t

Re: [I] `StrSubstitutor` cannot parse nested braces (logging-log4j2)

2024-07-05 Thread via GitHub
ppkarwasz commented on issue #2679: URL: https://github.com/apache/logging-log4j2/issues/2679#issuecomment-2210407353 To be more precise, the `StrLookup` parses the string `${env:TZ:-%d{-MM-dd HH:mm:ss}{GMT+00}}` as: 1. A lookup expression (`${env:TZ:-%d{-MM-dd HH:mm:ss}`), w