ppkarwasz commented on code in PR #2392:
URL: https://github.com/apache/logging-log4j2/pull/2392#discussion_r1535371587
##
log4j-api/src/main/java/org/apache/logging/log4j/util/Strings.java:
##
@@ -288,6 +289,19 @@ public static String trimToNull(final String str) {
ret
ppkarwasz commented on code in PR #2392:
URL: https://github.com/apache/logging-log4j2/pull/2392#discussion_r1535369617
##
log4j-api/src/main/java/org/apache/logging/log4j/message/DefaultFlowMessageFactory.java:
##
@@ -51,17 +52,22 @@ public DefaultFlowMessageFactory() {
*
dependabot[bot] opened a new pull request, #143:
URL: https://github.com/apache/logging-parent/pull/143
Bumps [github/codeql-action](https://github.com/github/codeql-action) from
3.24.8 to 3.24.9.
Commits
https://github.com/github/codeql-action/commit/1b1aada464948af03b950897e5
dependabot[bot] opened a new pull request, #144:
URL: https://github.com/apache/logging-parent/pull/144
Bumps
[dependabot/fetch-metadata](https://github.com/dependabot/fetch-metadata) from
1.6.0 to 2.0.0.
Release notes
Sourced from https://github.com/dependabot/fetch-metadata/rele
github-actions[bot] merged PR #144:
URL: https://github.com/apache/logging-parent/pull/144
--
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
ppkarwasz commented on code in PR #2392:
URL: https://github.com/apache/logging-log4j2/pull/2392#discussion_r1535393114
##
log4j-api-test/src/main/java/org/apache/logging/log4j/test/junit/Resources.java:
##
@@ -20,11 +20,36 @@
/**
* Constants to use the {@link ResourceLock}
ppkarwasz commented on code in PR #2393:
URL: https://github.com/apache/logging-log4j2/pull/2393#discussion_r1535402688
##
log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterFormatter.java:
##
@@ -240,12 +244,14 @@ static void formatMessage(
return;
github-actions[bot] merged PR #2399:
URL: https://github.com/apache/logging-log4j2/pull/2399
--
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: notificatio
ppkarwasz merged PR #2392:
URL: https://github.com/apache/logging-log4j2/pull/2392
--
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
ppkarwasz commented on PR #2392:
URL: https://github.com/apache/logging-log4j2/pull/2392#issuecomment-2015218299
@vy,
I am merging this PR in order to be able to use `2.24.0-SNAPSHOT` versions
in the PRs concerning the `main` branch.
Please let me know if you find something els
SeasonPanPan commented on code in PR #2393:
URL: https://github.com/apache/logging-log4j2/pull/2393#discussion_r1535761095
##
log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterFormatter.java:
##
@@ -240,12 +244,14 @@ static void formatMessage(
return
erikmav commented on code in PR #129:
URL: https://github.com/apache/logging-log4net/pull/129#discussion_r1535848543
##
src/log4net.Tests/DateFormatter/DateTimeDateFormatterTest.cs:
##
@@ -0,0 +1,59 @@
+#region Apache License
+//
+// Licensed to the Apache Software Foundation (A
andreycha commented on issue #118:
URL:
https://github.com/apache/logging-log4net/issues/118#issuecomment-2015536188
Hi @FreeAndNil any ETA when 2.0.17 stable version is going to be released? I
need some changed from 2.0.16 but don't want to upgrade without this hotfix.
--
This is an aut
FreeAndNil commented on issue #118:
URL:
https://github.com/apache/logging-log4net/issues/118#issuecomment-2015577460
@andreycha definitely next week.
--
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 g
marcel-landman opened a new issue, #130:
URL: https://github.com/apache/logging-log4net/issues/130
This is where we are told the documentation is:
https://logging.apache.org/log4net/
All the top level links work, but as soon as you click through to a second
level, the link is broke
FreeAndNil commented on PR #121:
URL: https://github.com/apache/logging-log4net/pull/121#issuecomment-2015886339
Fixes #130
--
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.
FreeAndNil commented on issue #130:
URL:
https://github.com/apache/logging-log4net/issues/130#issuecomment-2015887248
@marcel-landman thanks for reporting.
I discovered the same yesterday 😉.
A fix
(https://github.com/apache/logging-log4net/pull/121/commits/61e28326da40646161fe5167905f
FreeAndNil closed issue #130: Most links in the documentation are broken
URL: https://github.com/apache/logging-log4net/issues/130
--
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
FreeAndNil merged PR #129:
URL: https://github.com/apache/logging-log4net/pull/129
--
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
SeasonPanPan commented on code in PR #2393:
URL: https://github.com/apache/logging-log4j2/pull/2393#discussion_r1536625743
##
log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterFormatter.java:
##
@@ -240,12 +244,14 @@ static void formatMessage(
return
dependabot[bot] opened a new pull request, #145:
URL: https://github.com/apache/logging-parent/pull/145
Bumps
[org.cyclonedx:cyclonedx-maven-plugin](https://github.com/CycloneDX/cyclonedx-maven-plugin)
from 2.7.11 to 2.8.0.
Release notes
Sourced from https://github.com/CycloneDX/c
dependabot[bot] opened a new pull request, #146:
URL: https://github.com/apache/logging-parent/pull/146
Bumps org.apache.logging.log4j:log4j-changelog-maven-plugin from 0.7.0 to
0.8.0.
[ und
vy commented on PR #2400:
URL: https://github.com/apache/logging-log4j2/pull/2400#issuecomment-2017490303
> Since the only current user of `Recycler` is Log4j Core 3.x, we removed
the static recycler registry and replaced it with an entry in `DefaultBundle`.
This is not correct. There
ppkarwasz opened a new issue, #2401:
URL: https://github.com/apache/logging-log4j2/issues/2401
Currently instances of `MutableLogEvent` are prevented from traveling
between threads. In 2.x the reason was simple: the event was bound to a
`ThreadLocal` and was cleared upon the return from the
ppkarwasz commented on code in PR #2400:
URL: https://github.com/apache/logging-log4j2/pull/2400#discussion_r1537344241
##
log4j-kit/src/main/java/org/apache/logging/log4j/kit/recycler/internal/ThreadLocalRecyclerFactoryProvider.java:
##
Review Comment:
This remark certainl
ppkarwasz commented on PR #2400:
URL: https://github.com/apache/logging-log4j2/pull/2400#issuecomment-2017657169
> > Since the only current user of `Recycler` is Log4j Core 3.x, we removed
the static recycler registry and replaced it with an entry in `DefaultBundle`.
>
> This is not c
vy opened a new pull request, #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404
This PR adds the following new steps to the project build.
1. At `compile` phase, auto-generate plugin descriptors using an annotation
processor
2. At `site` phase, using the generated plug
ppkarwasz commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537388385
##
log4j-api/pom.xml:
##
@@ -28,7 +28,7 @@
Apache Log4j API
The Apache Log4j API
-false
+false
Review Comment:
This PR also replaces our
vy commented on code in PR #141:
URL: https://github.com/apache/logging-parent/pull/141#discussion_r1537393539
##
pom.xml:
##
@@ -1659,6 +1667,49 @@ import org.apache.commons.codec.digest.*;
+
+
+
+ coverage
+
+
+
+
+
+o
vy commented on code in PR #141:
URL: https://github.com/apache/logging-parent/pull/141#discussion_r1537393539
##
pom.xml:
##
@@ -1659,6 +1667,49 @@ import org.apache.commons.codec.digest.*;
+
+
+
+ coverage
+
+
+
+
+
+o
vy commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537396203
##
log4j-api/pom.xml:
##
@@ -28,7 +28,7 @@
Apache Log4j API
The Apache Log4j API
-false
+false
Review Comment:
Yes, and this is in line with `ma
ppkarwasz commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537394035
##
log4j-core-java9/pom.xml:
##
@@ -30,6 +30,8 @@
9
true
+true
Review Comment:
The `log4j-core-java9` artifact was intentionally "install
ppkarwasz merged PR #147:
URL: https://github.com/apache/logging-parent/pull/147
--
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.
ppkarwasz closed issue #105: Clean up after next `cyclonedx-maven-plugin`
upgrade
URL: https://github.com/apache/logging-parent/issues/105
--
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 specifi
vy commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537473129
##
log4j-core/pom.xml:
##
@@ -222,40 +233,61 @@
+
maven-compiler-plugin
-
-
-
+
+
vy commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537472842
##
log4j-core-java9/pom.xml:
##
@@ -30,6 +30,8 @@
9
true
+true
Review Comment:
Made it `./mvnw install`able again in
a4851854936aea9ebc862b12e4
github-actions[bot] merged PR #2405:
URL: https://github.com/apache/logging-log4j2/pull/2405
--
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: notificatio
vy commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537481147
##
log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/xml/XmlSchemaTest.java:
##
@@ -1,103 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation
ppkarwasz commented on code in PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404#discussion_r1537484589
##
log4j-api/pom.xml:
##
@@ -28,7 +28,7 @@
Apache Log4j API
The Apache Log4j API
-false
+false
Review Comment:
No, it is just a note for
ppkarwasz merged PR #141:
URL: https://github.com/apache/logging-parent/pull/141
--
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.
ppkarwasz closed issue #140: Add profile to generate a coverage report
URL: https://github.com/apache/logging-parent/issues/140
--
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.
vy opened a new pull request, #2407:
URL: https://github.com/apache/logging-log4j2/pull/2407
This PR ports #2404 from `2.x` to `main` for Log4j 3.
--
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
ppkarwasz opened a new pull request, #2408:
URL: https://github.com/apache/logging-log4j2/pull/2408
Due to differences in the lifecycle of Log4j Core and Kubernetes Client, we
remove `log4j-kubernetes` from the 3.x release and redirect users to Fabric8's
own `kubernetes-log4j` artifact intr
ppkarwasz commented on code in PR #2407:
URL: https://github.com/apache/logging-log4j2/pull/2407#discussion_r1537565796
##
log4j-parent/pom.xml:
##
@@ -1028,6 +1031,118 @@
+
+
+
+ plugin-processing
+
+
+
+
+ .log4j-pl
ppkarwasz commented on PR #2391:
URL: https://github.com/apache/logging-log4j2/pull/2391#issuecomment-2017998260
@vy, @garydgregory,
As I explained in #2389, I am for closing this PR and look for a different
way to solve that issue.
--
This is an automated message from the Apache G
fluffynuts merged PR #121:
URL: https://github.com/apache/logging-log4net/pull/121
--
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
FreeAndNil commented on issue #118:
URL:
https://github.com/apache/logging-log4net/issues/118#issuecomment-2018303353
@andreycha
https://github.com/apache/logging-log4net/releases/tag/rel%2F2.0.17
https://www.nuget.org/packages/log4net/2.0.17
--
This is an automated message from the
andreycha commented on issue #118:
URL:
https://github.com/apache/logging-log4net/issues/118#issuecomment-2018333051
@FreeAndNil thanks a lot!
--
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
erikmav opened a new pull request, #131:
URL: https://github.com/apache/logging-log4net/pull/131
Enable nullable for the whole repo, including enabling some disabled XML
tests and adding more test coverage. For #124 .
--
This is an automated message from the Apache Git Service.
To respon
dependabot[bot] opened a new pull request, #132:
URL: https://github.com/apache/logging-log4j-audit/pull/132
Bumps commons-logging:commons-logging from 1.3.0 to 1.3.1.
[
from 1.9.21.2 to 1.9.22.
Release notes
Sourced from https://github.com/eclipse/org.aspectj/releases
github-actions[bot] merged PR #133:
URL: https://github.com/apache/logging-log4j-audit/pull/133
--
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: notifica
github-actions[bot] merged PR #133:
URL: https://github.com/apache/logging-log4j-audit/pull/133
--
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: notifica
github-actions[bot] merged PR #132:
URL: https://github.com/apache/logging-log4j-audit/pull/132
--
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: notifica
vy closed pull request #2391: Adds new `NoSqlConnection#insertObject()`
accepting `LogEvent`
URL: https://github.com/apache/logging-log4j2/pull/2391
--
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 t
vy commented on code in PR #2407:
URL: https://github.com/apache/logging-log4j2/pull/2407#discussion_r1538142217
##
log4j-parent/pom.xml:
##
@@ -1028,6 +1031,118 @@
+
+
+
+ plugin-processing
+
+
+
+
+ .log4j-plugin-pr
vy commented on code in PR #2407:
URL: https://github.com/apache/logging-log4j2/pull/2407#discussion_r1538142217
##
log4j-parent/pom.xml:
##
@@ -1028,6 +1031,118 @@
+
+
+
+ plugin-processing
+
+
+
+
+ .log4j-plugin-pr
dependabot[bot] opened a new pull request, #89:
URL: https://github.com/apache/logging-log4j-transform/pull/89
Bumps org.ow2.asm:asm-bom from 9.6 to 9.7.
[
from 6.9.2 to 6.11.0.
Release notes
Sourced from https://github.com/fabric8io/kub
dependabot[bot] commented on PR #106:
URL:
https://github.com/apache/logging-log4j-samples/pull/106#issuecomment-2019174806
Superseded by #120.
--
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 th
dependabot[bot] closed pull request #106: Bump io.fabric8:kubernetes-client-bom
from 6.9.2 to 6.10.0
URL: https://github.com/apache/logging-log4j-samples/pull/106
--
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
ppkarwasz opened a new pull request, #2410:
URL: https://github.com/apache/logging-log4j2/pull/2410
This PR depends on #2400 being accepted.
We rebase Log4j Core 3.x and its submodules on top of Log4j API 2.x, which
concludes #2290.
## Main changes
- Classes moved to Lo
ppkarwasz opened a new pull request, #2411:
URL: https://github.com/apache/logging-log4j2/pull/2411
This PR is based on #2410.
Since the Log4j API 3.x modules are no longer used, they are simply deleted.
--
This is an automated message from the Apache Git Service.
To respond to
vy merged PR #2404:
URL: https://github.com/apache/logging-log4j2/pull/2404
--
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
vy merged PR #2407:
URL: https://github.com/apache/logging-log4j2/pull/2407
--
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
vy commented on PR #2408:
URL: https://github.com/apache/logging-log4j2/pull/2408#issuecomment-2019921087
+1 for dropping it from `2.x`. @ppkarwasz, could you also submit a PR for
that too, please?
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
ppkarwasz opened a new pull request, #2412:
URL: https://github.com/apache/logging-log4j2/pull/2412
Due to differences in the lifecycle of Log4j Core and Kubernetes Client, we
remove `log4j-kubernetes` from the 2.x release and redirect users to Fabric8's
own `kubernetes-log4j` artifact intr
ppkarwasz commented on PR #2408:
URL: https://github.com/apache/logging-log4j2/pull/2408#issuecomment-2019997058
@vy, I submitted a 2.x equivalent in #2412
--
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
jolly28 opened a new issue, #2414:
URL: https://github.com/apache/logging-log4j2/issues/2414
## Description
My java:17.0.5 Application is based on struts2-core:6.3.0.2 and I am using
the log4j-api:2.17.2 as the dependency and the application is running fine.
When log4j-api uplifted to
vy commented on issue #2075:
URL:
https://github.com/apache/logging-log4j2/issues/2075#issuecomment-2020177670
This issue is (mostly) fixed by #2104:
* `QueueFactories` and `QueueFactory` are removed
* `BlockingQueueFactory` is left in `log4j-core` with a JCTools-based
implementat
vy closed issue #2075: Merge queue factory interfaces
URL: https://github.com/apache/logging-log4j2/issues/2075
--
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,
github-actions[bot] merged PR #2415:
URL: https://github.com/apache/logging-log4j2/pull/2415
--
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: notificatio
github-actions[bot] merged PR #2403:
URL: https://github.com/apache/logging-log4j2/pull/2403
--
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: notificatio
github-actions[bot] merged PR #2413:
URL: https://github.com/apache/logging-log4j2/pull/2413
--
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: notificatio
FreeAndNil merged PR #100:
URL: https://github.com/apache/logging-log4net/pull/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 comment.
To unsubscribe, e-mail: notifications-unsubsc
vy commented on code in PR #2396:
URL: https://github.com/apache/logging-log4j2/pull/2396#discussion_r1539971265
##
log4j-1.2-api/src/main/java/org/apache/log4j/config/PropertiesConfigurationFactory.java:
##
@@ -45,18 +45,26 @@ public class PropertiesConfigurationFactory extends
ppkarwasz commented on code in PR #2396:
URL: https://github.com/apache/logging-log4j2/pull/2396#discussion_r1540045999
##
log4j-1.2-api/src/main/java/org/apache/log4j/config/PropertiesConfigurationFactory.java:
##
@@ -45,18 +45,26 @@ public class PropertiesConfigurationFactory
FreeAndNil closed issue #127: Enable building log4net with docker container
URL: https://github.com/apache/logging-log4net/issues/127
--
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 comm
201 - 300 of 14006 matches
Mail list logo
