[PR] Fix logger typo 'patterrn' -> 'pattern' [logging-log4j2]

2025-04-15 Thread via GitHub
hyunw9 opened a new pull request, #3617: URL: https://github.com/apache/logging-log4j2/pull/3617 Description Corrected a typo in error message for invalid conversion pattern in LogLog. Changed patterrn → pattern for better readability. found typo : ![화면 캡처 2025-04-16 075649]

[PR] cleanup changes for Antora (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil opened a new pull request, #238: URL: https://github.com/apache/logging-log4net/pull/238 (no comment) -- 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 unsubscr

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045351878 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045351528 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045350948 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045350487 ## src/site/antora/antora.yml: ## @@ -0,0 +1,50 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements.

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2045316770 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045319737 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2045316770 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045307641 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045307641 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045302745 ## src/site/antora/antora.yml: ## @@ -0,0 +1,50 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045299948 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045298863 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
FreeAndNil commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045298557 ## antora-playbook.yaml: ## @@ -0,0 +1,141 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045294226 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045293724 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045290659 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045290332 ## pom.xml: ## @@ -39,157 +58,131 @@ https://mail-archives.apache.org/mod_mbox/logging-commits/ - - - Apache License, Version 2.0 - h

Re: [PR] Migrate log4net site to Antora and use logging-parent pom (logging-log4net)

2025-04-15 Thread via GitHub
vy commented on code in PR #210: URL: https://github.com/apache/logging-log4net/pull/210#discussion_r2045288723 ## antora-playbook.yaml: ## @@ -0,0 +1,141 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTI

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2045176799 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled":

Re: [PR] Bump apache/logging-parent/.github/workflows/deploy-site-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 [logging-log4j-jmx-gui]

2025-04-15 Thread via GitHub
ppkarwasz closed pull request #28: Bump apache/logging-parent/.github/workflows/deploy-site-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 URL: https://github.com/apache/logging-log4j-jmx-gui/pull/28 -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] Adapt `merge-dependabot` to RTC [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on code in PR #3603: URL: https://github.com/apache/logging-log4j2/pull/3603#discussion_r2044458768 ## .github/workflows/add-dependabot-changelog.yaml: ## Review Comment: Yes, it should, be we need to test it first. The hard part is probably to trigger

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044567613 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Bump apache/logging-parent/.github/workflows/merge-dependabot-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 [logging-log4j-tools]

2025-04-15 Thread via GitHub
ppkarwasz closed pull request #172: Bump apache/logging-parent/.github/workflows/merge-dependabot-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 URL: https://github.com/apache/logging-log4j-tools/pull/172 -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Bump apache/logging-parent/.github/workflows/build-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 [logging-log4j-kotlin]

2025-04-15 Thread via GitHub
ppkarwasz closed pull request #106: Bump apache/logging-parent/.github/workflows/build-reusable.yaml@rel/11.3.0 from 11.3.0 to 12.0.0 URL: https://github.com/apache/logging-log4j-kotlin/pull/106 -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Fix `SEI MET07-J` violations [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz merged PR #3602: URL: https://github.com/apache/logging-log4j2/pull/3602 -- 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: [PR] Lock Antora dependencies [logging-parent]

2025-04-15 Thread via GitHub
vy commented on code in PR #367: URL: https://github.com/apache/logging-parent/pull/367#discussion_r2043956202 ## github.json: ## Review Comment: What is this and why is this needed? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [I] Fix `SEI MET07-J` violations [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz closed issue #3601: Fix `SEI MET07-J` violations URL: https://github.com/apache/logging-log4j2/issues/3601 -- 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 unsubscr

Re: [PR] Fix `SEI MET07-J` violations [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #3602: URL: https://github.com/apache/logging-log4j2/pull/3602#issuecomment-2804135956 @vy, thanks! :100: -- 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] Reintroduce the Scorecard workflow [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on issue #3604: URL: https://github.com/apache/logging-log4j2/issues/3604#issuecomment-2804252632 We had enabled Scorecards, and it helped with 1. Creating a maintenance burden (Remember how many times we needed to fix its CI workflow?) 2. Bringing literally no value

[PR] Fix Sponsors buttons [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz opened a new pull request, #3616: URL: https://github.com/apache/logging-log4j2/pull/3616 Due to limitations in the number of GitHub Sponsors buttons (there can be only up to 4 buttons), part of the team is ignored. This replaces the buttons with our "Sponsorship" page and a

Re: [PR] Lock Antora dependencies [logging-parent]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #367: URL: https://github.com/apache/logging-parent/pull/367#issuecomment-280480 I modified the way `node_modules` are cached: since running a second `actions/checkout` in the same workflow deletes `node_modules`, the folder must be restored at the end of the w

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044531747 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Add "Sponsor" buttons to repository [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #3580: URL: https://github.com/apache/logging-log4j2/pull/3580#issuecomment-2805024558 Fixed in #3616 -- 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 comme

Re: [PR] Upgrade Gradle and Wrapper [logging-log4j-samples]

2025-04-15 Thread via GitHub
ppkarwasz merged PR #310: URL: https://github.com/apache/logging-log4j-samples/pull/310 -- 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-un

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044531747 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Move JSpecify from `provided` to `compile` scope [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #3228: URL: https://github.com/apache/logging-log4j2/pull/3228#issuecomment-2805698294 > will this have any [backward compatibility] implications on the fat JAR users? I am not sure what implications are you thinking about. Yes, their fat JARs will be a coup

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044531747 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044567613 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": tr

Re: [PR] Bump com.android.application from 8.2.0 to 8.9.1 [logging-log4j-samples]

2025-04-15 Thread via GitHub
github-actions[bot] merged PR #315: URL: https://github.com/apache/logging-log4j-samples/pull/315 -- 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: notifi

Re: [PR] Adapt `merge-dependabot` to RTC [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #3603: URL: https://github.com/apache/logging-log4j2/pull/3603#issuecomment-2804944691 Currently all Dependabot workflows fail (see [job 40576197611](https://github.com/apache/logging-log4j2/actions/runs/14467978596/job/40576197611) and the new commit does not trig

Re: [I] Reintroduce the Scorecard workflow [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz commented on issue #3604: URL: https://github.com/apache/logging-log4j2/issues/3604#issuecomment-2804927362 > > we have more control on what the public sees > > Do you imply that OSSF is manipulating the scores? Or, we can manipulate the scores as we see fit? No, but

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044010077 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -37,8 +42,9 @@ } }, "logging.googleapis.com/insertId": { -"$resolver": "counter"

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
ViliusS commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044317642 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -37,8 +42,9 @@ } }, "logging.googleapis.com/insertId": { -"$resolver": "cou

Re: [PR] Adapt `merge-dependabot` to RTC [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on code in PR #3603: URL: https://github.com/apache/logging-log4j2/pull/3603#discussion_r2044019381 ## .github/workflows/add-dependabot-changelog.yaml: ## Review Comment: Shouldn't this be a reusable in `logging-parent`? Does every project need to roll out the

Re: [PR] Use workflows on the same branch [logging-parent]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #379: URL: https://github.com/apache/logging-parent/pull/379#issuecomment-2804521321 Since changes to reusable workflows are half of the PRs in this repository, I find that this is not just a convenience for maintainers: - A PR can be approved by mistake, bec

Re: [PR] Add "Sponsor" buttons to repository [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on PR #3580: URL: https://github.com/apache/logging-log4j2/pull/3580#issuecomment-2804618550 @ppkarwasz, putting me aside, even @rgoers is excluded. Would you mind submitting a PR reverting this change, please? -- This is an automated message from the Apache Git Service. To r

Re: [PR] Lock Antora dependencies [logging-parent]

2025-04-15 Thread via GitHub
ppkarwasz commented on code in PR #367: URL: https://github.com/apache/logging-parent/pull/367#discussion_r2044218634 ## github.json: ## Review Comment: Nice catch! :100: This was a temporary file with `${{ github.toString() }}` I created to see if the branch name o

[PR] Enforce RTC [logging-log4j-samples]

2025-04-15 Thread via GitHub
ppkarwasz opened a new pull request, #320: URL: https://github.com/apache/logging-log4j-samples/pull/320 This is an implementation of the [PMC vote in `dev@logging`](https://lists.apache.org/thread/8ffdw1zqrvwrsdr0ng0m2rgtdnpzg4hk). I didn't add any required checks, since: - Any test

Re: [PR] Bump com.android.application from 8.2.0 to 8.9.1 [logging-log4j-samples]

2025-04-15 Thread via GitHub
ppkarwasz commented on PR #315: URL: https://github.com/apache/logging-log4j-samples/pull/315#issuecomment-2804525111 @dependabot rebase, the build should succeed now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] Pass arguments to Gradle via environment variables [logging-log4j-samples]

2025-04-15 Thread via GitHub
ppkarwasz merged PR #314: URL: https://github.com/apache/logging-log4j-samples/pull/314 -- 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-un

Re: [PR] Remove Apache Snapshots from release builds [logging-log4j-samples]

2025-04-15 Thread via GitHub
ppkarwasz merged PR #313: URL: https://github.com/apache/logging-log4j-samples/pull/313 -- 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-un

Re: [PR] Use workflows on the same branch [logging-parent]

2025-04-15 Thread via GitHub
vy commented on PR #379: URL: https://github.com/apache/logging-parent/pull/379#issuecomment-2804224837 Although I can see the advantage (i.e., a PR touching to reusable workflows will directly get employed, instead of the one at `main`), I'm not keen on with this change for the following r

Re: [I] Fix `SEI MET07-J` violations [logging-log4j2]

2025-04-15 Thread via GitHub
ppkarwasz closed issue #3601: Fix `SEI MET07-J` violations URL: https://github.com/apache/logging-log4j2/issues/3601 -- 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 unsubscr

Re: [PR] Move JSpecify from `provided` to `compile` scope [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on PR #3228: URL: https://github.com/apache/logging-log4j2/pull/3228#issuecomment-2804313789 @ppkarwasz, will this have any [backward compatibility] implications on the fat JAR users? -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] Adjust GcpLayout JSON to latest format [logging-log4j2]

2025-04-15 Thread via GitHub
vy commented on code in PR #3586: URL: https://github.com/apache/logging-log4j2/pull/3586#discussion_r2044006120 ## log4j-layout-template-json/src/main/resources/GcpLayout.json: ## @@ -49,25 +55,15 @@ "key": "span_id" }, "logging.googleapis.com/trace_sampled": true, -