carterkozak commented on pull request #730:
URL: https://github.com/apache/logging-log4j2/pull/730#issuecomment-1022497645
I defer to @garydgregory for Tomact knowledge, but I'd question bumping a
minimum dependency constraint two major versions in a patch release.
--
This is an automate
ppkarwasz commented on pull request #730:
URL: https://github.com/apache/logging-log4j2/pull/730#issuecomment-1022528780
The only Tomcat class used by Log4j is `org.apache.juli.logging.Log`, which
has not been modified in ages (cf.
[Github](https://github.com/apache/tomcat/blob/main/java/o
rm5248 commented on pull request #106:
URL: https://github.com/apache/logging-log4cxx/pull/106#issuecomment-1022725569
Merging to master does produce a number of(expected) bugs:
https://sonarcloud.io/summary/overall?id=log4cxx
It's scanning a bunch of files that we don't care about a
ppkarwasz opened a new pull request #731:
URL: https://github.com/apache/logging-log4j2/pull/731
The `TomcatLookup` can be used to retrieve the engine, host and context
names under which the application is running. These are also provided by the
original Tomcat JULI implementation (cf.
[C
vy commented on pull request #690:
URL: https://github.com/apache/logging-log4j2/pull/690#issuecomment-1022966398
Really sharp of you @jvz! :hocho: Done.
--
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 t
vy merged pull request #728:
URL: https://github.com/apache/logging-log4j2/pull/728
--
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-
rdotterer09 commented on a change in pull request #722:
URL: https://github.com/apache/logging-log4j2/pull/722#discussion_r792815957
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/Property.java
##
@@ -57,6 +63,14 @@ public String getName() {
vy merged pull request #729:
URL: https://github.com/apache/logging-log4j2/pull/729
--
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-
vy commented on pull request #729:
URL: https://github.com/apache/logging-log4j2/pull/729#issuecomment-1023334369
Thanks so much @ppkarwasz!
--
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 s
avandeursen commented on a change in pull request #731:
URL: https://github.com/apache/logging-log4j2/pull/731#discussion_r793929483
##
File path:
log4j-appserver/src/main/java/org/apache/logging/log4j/appserver/tomcat/TomcatLookup.java
##
@@ -0,0 +1,67 @@
+/*
+ * Licensed to
ppkarwasz commented on a change in pull request #731:
URL: https://github.com/apache/logging-log4j2/pull/731#discussion_r793958070
##
File path:
log4j-appserver/src/main/java/org/apache/logging/log4j/appserver/tomcat/TomcatLookup.java
##
@@ -0,0 +1,67 @@
+/*
+ * Licensed to th
carterkozak opened a new pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732
Previously lookups between configuration properties weren't evaluated
until they were used in the configuration, which resulted in
behavior more prohibitive for RoutingAppenders than anti
jvz opened a new pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733
This adds a properties format feature where the property names "rootLogger"
and "logger." can be set to a level and one or more appender refs.
https://issues.apache.org/jira/browse/LOG4J2-3341
dependabot[bot] opened a new pull request #734:
URL: https://github.com/apache/logging-log4j2/pull/734
Bumps `mockitoVersion` from 4.2.0 to 4.3.1.
Updates `mockito-core` from 4.2.0 to 4.3.1
Release notes
Sourced from https://github.com/mockito/mockito/releases";>mockito-core's
Apache9 commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794212770
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794221405
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794221405
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
rgoers commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794225373
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (A
rgoers commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794225373
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (A
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794225556
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794226178
##
File path: log4j-core/src/test/resources/RootLoggerLevelAppenderTest.properties
##
@@ -0,0 +1,21 @@
+#
+# Licensed to the Apache Software Foundation (
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794225556
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
ppkarwasz commented on pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#issuecomment-1023935035
@jvz: maybe you can add a default for all component names: `name =
key.replaceAll("-", ".")`? This way you'll allow also to configure a logger in
one property:
`l
Apache9 commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794282859
##
File path: log4j-core/src/test/resources/LoggerLevelAppenderTest.properties
##
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation (
pseiderer opened a new pull request #107:
URL: https://github.com/apache/logging-log4cxx/pull/107
- only check for boost if really needed (in case of legacy c++ standard <
c++17)
Signed-off-by: Peter Seiderer
---
Notes:
- fixes the following buildroot configure/compil
carterkozak commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794523439
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17
carterkozak commented on a change in pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#discussion_r794552367
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/PropertiesPlugin.java
##
@@ -45,15 +46,14 @@ private PropertiesPlugi
carterkozak commented on a change in pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#discussion_r794553848
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/LookupResult.java
##
@@ -0,0 +1,35 @@
+/*
+ * Licensed to the Apache
garydgregory commented on pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#issuecomment-1024309838
Maybe this is the wrong place to ask but it feels to me we are going about
this backward or I am missing some information. I would find it helpful to work
from the do
vorburger closed pull request #700:
URL: https://github.com/apache/logging-log4j2/pull/700
--
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: notific
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794643584
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17 @@ publi
jvz commented on pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#issuecomment-1024367931
> @jvz: maybe you can add a default for all component names: `name =
key.replaceAll("-", ".")`? This way you'll allow also to configure a logger in
one property:
>
> `l
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794645183
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17 @@ publi
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794647949
##
File path: log4j-core/src/test/resources/RootLoggerLevelAppenderTest.properties
##
@@ -0,0 +1,21 @@
+#
+# Licensed to the Apache Software Foundation (
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794652735
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17 @@ publi
carterkozak commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794655910
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17
jvz commented on a change in pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#discussion_r794657514
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/config/properties/PropertiesConfigurationBuilder.java
##
@@ -190,6 +193,17 @@ publi
carterkozak commented on pull request #717:
URL: https://github.com/apache/logging-log4j2/pull/717#issuecomment-1024420037
committed as fc31b44b629d9fe46bd96bfd02382519dfdb5f40
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
carterkozak closed pull request #717:
URL: https://github.com/apache/logging-log4j2/pull/717
--
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: notif
carterkozak commented on pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#issuecomment-1024613948
This experiment came out of a discussion with Ralph, and deserves more
context in this PR. This is still marked as a draft, and needs a better
description of intent an
carterkozak commented on pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#issuecomment-1024737180
I have updated the description with several more words :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
jvz commented on pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#issuecomment-1024803593
I've squashed what's been reviewed. Will add more commits for docs and
changelog.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
jvz merged pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733
--
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
jvz commented on pull request #729:
URL: https://github.com/apache/logging-log4j2/pull/729#issuecomment-1024824326
Noticed this is needed in master as well in order to support the new
properties format features.
--
This is an automated message from the Apache Git Service.
To respond to t
ppkarwasz commented on pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#issuecomment-1024854364
@jvz: #729 drops previously invalid entries like
`logger.core-config-properties=INFO,app1,app2`. You should probably add a test
for that.
--
This is an automated mess
jvz commented on pull request #733:
URL: https://github.com/apache/logging-log4j2/pull/733#issuecomment-1024854475
I have some code in PropertiesConfigurationBuilder that manually includes
the invalid keys left behind.
--
This is an automated message from the Apache Git Service.
To respo
jvz commented on pull request #727:
URL: https://github.com/apache/logging-log4j2/pull/727#issuecomment-1024861520
@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
jvz commented on pull request #695:
URL: https://github.com/apache/logging-log4j2/pull/695#issuecomment-1024861817
@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
jvz commented on pull request #727:
URL: https://github.com/apache/logging-log4j2/pull/727#issuecomment-1024861926
@dependabot recreate
--
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 specif
dependabot[bot] commented on pull request #695:
URL: https://github.com/apache/logging-log4j2/pull/695#issuecomment-1024862206
OK, I won't notify you about version 4.x.x again, unless you re-open this PR
or update to a 4.x.x release yourself.
--
This is an automated message from the Apac
dependabot[bot] closed pull request #695:
URL: https://github.com/apache/logging-log4j2/pull/695
--
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: n
jvz commented on pull request #695:
URL: https://github.com/apache/logging-log4j2/pull/695#issuecomment-1024862185
@dependabot ignore this major version
Breaking API changes. Probably needs a new module.
--
This is an automated message from the Apache Git Service.
To respond to the
ppkarwasz opened a new pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735
As an extension of #733, this PR allows to specify the name of any component
in its key. Dots in components names are escaped using dashes `-`. Since this
is not the only way to specify a compo
garydgregory commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1024942878
Do we really want to open the door here? What about underscore? Any valid
Java name separators? What about mixing separators?
--
This is an automated message from
carterkozak commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1024944765
I agree with Gary. I’m concerned about the user experience, when an operator
sees logger.org-apache-foo it’s not going to be obvious that is equivalent to
org.apache.f
ronosaurus opened a new pull request #736:
URL: https://github.com/apache/logging-log4j2/pull/736
DO NOT MERGE: candidate for a 2.19 release after more important changes
occur in 2.18
```
// logEvents.html.zip
smtpBuilder.setAttachEvents(true);
smtpBuilder.setAttachEventsCo
rm5248 merged pull request #107:
URL: https://github.com/apache/logging-log4cxx/pull/107
--
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: notificat
rm5248 commented on pull request #107:
URL: https://github.com/apache/logging-log4cxx/pull/107#issuecomment-1024955174
I'm surprised that it failed - I thought that the try_compile tests would
not cause it to completely fail. Maybe I just don't understand enough about
how it works.
--
ppkarwasz commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1024956766
What about a more common way of escaping the dots, like `\.`? Or maybe
quoting/double-quoting the logger name? Although it might not be intuitive why
the part after `log
rm5248 commented on a change in pull request #102:
URL: https://github.com/apache/logging-log4cxx/pull/102#discussion_r795079918
##
File path: src/main/cpp/rollingpolicybase.cpp
##
@@ -38,6 +39,7 @@ IMPLEMENT_LOG4CXX_OBJECT(RollingPolicyBase)
RollingPolicyBase::RollingPolicy
jvz commented on a change in pull request #736:
URL: https://github.com/apache/logging-log4j2/pull/736#discussion_r795089083
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/net/SmtpManager.java
##
@@ -16,24 +16,16 @@
*/
package org.apache.logging.log4j.
ppkarwasz edited a comment on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1024956766
What about a more common way of escaping the dots, like `\.`? Or maybe
quoting/double-quoting the logger name? Although it might not be intuitive why
the part aft
ppkarwasz edited a comment on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1024956766
What about a more common way of escaping the dots, like `\.`? Or maybe
quoting/double-quoting the logger name? Although it might not be intuitive why
the part aft
ppkarwasz opened a new pull request #737:
URL: https://github.com/apache/logging-log4j2/pull/737
The properties of the generated appenders are not always the same as in
Log4j 1.x. This PR:
- adds default values to all Log4j 1.x configuration builders,
- adds a couple of properti
rdotterer09 commented on pull request #717:
URL: https://github.com/apache/logging-log4j2/pull/717#issuecomment-1025161115
thanks for the help
--
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
rdotterer09 removed a comment on pull request #717:
URL: https://github.com/apache/logging-log4j2/pull/717#issuecomment-1025161115
thanks for the help
--
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
dependabot[bot] opened a new pull request #738:
URL: https://github.com/apache/logging-log4j2/pull/738
Bumps `xmlunitVersion` from 2.8.4 to 2.9.0.
Updates `xmlunit-core` from 2.8.4 to 2.9.0
Release notes
Sourced from https://github.com/xmlunit/xmlunit/releases";>xmlunit-core's
vy merged pull request #730:
URL: https://github.com/apache/logging-log4j2/pull/730
--
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-
dependabot[bot] closed pull request #662:
URL: https://github.com/apache/logging-log4j2/pull/662
--
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: n
dependabot[bot] commented on pull request #662:
URL: https://github.com/apache/logging-log4j2/pull/662#issuecomment-1025695730
Looks like org.apache.tomcat:tomcat-catalina is no longer a dependency, so
this is no longer needed.
--
This is an automated message from the Apache Git Service.
carterkozak commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1026245393
I could see something like this working:
`logger.org.apache.foo.level = INFO, console`
If the property name begins with a known value `logger.` and ends with
dependabot[bot] opened a new pull request #739:
URL: https://github.com/apache/logging-log4j2/pull/739
Bumps
[maven-checkstyle-plugin](https://github.com/apache/maven-checkstyle-plugin)
from 3.0.0 to 3.1.2.
Commits
https://github.com/apache/maven-checkstyle-plugin/commit/27b9
ppkarwasz commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1026545182
@carterkozak: this could work, but maybe let's use another suffix for the
condensed property. Is `.config` Ok?
Basically it could work like this:
- the implici
ppkarwasz edited a comment on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1026545182
@carterkozak: this could work, but maybe let's use another suffix for the
condensed property. Is `.config` Ok?
Basically it could work like this:
- the
ppkarwasz edited a comment on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1026545182
@carterkozak: this could work, but maybe let's use another suffix for the
condensed property. Is `.config` Ok?
Basically it could work like this:
- the
ppkarwasz edited a comment on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1026545182
@carterkozak: this could work, but maybe let's use another suffix for the
condensed property. Is `.config` or `.value` Ok?
Basically it could work like this
carterkozak commented on pull request #735:
URL: https://github.com/apache/logging-log4j2/pull/735#issuecomment-1027030918
It sounds like we're discussing improving several different parts of the
properties configuration scheme at once -- It's great to have an idea of where
we want to go b
vy commented on pull request #731:
URL: https://github.com/apache/logging-log4j2/pull/731#issuecomment-1028077995
I am against introducing components for 3rd party software integrations.
These better be developed and maintained by the community; Log4j provides
first-class extensibility mec
vy merged pull request #734:
URL: https://github.com/apache/logging-log4j2/pull/734
--
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-
vy commented on pull request #737:
URL: https://github.com/apache/logging-log4j2/pull/737#issuecomment-1028089316
@garydgregory, would you be interested in taking this on as a part of your
ongoing `log4j-1.2-bridge` crusade?
--
This is an automated message from the Apache Git Service.
To
vy merged pull request #738:
URL: https://github.com/apache/logging-log4j2/pull/738
--
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-
vy merged pull request #739:
URL: https://github.com/apache/logging-log4j2/pull/739
--
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-
garydgregory commented on pull request #737:
URL: https://github.com/apache/logging-log4j2/pull/737#issuecomment-1028104019
@vy I might be able to take a look this weekend.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
sman-81 commented on pull request #542:
URL: https://github.com/apache/logging-log4j2/pull/542#issuecomment-1028218237
"we love pull requests" they say
you send them prs
you listen to their criticism
you incorporate their feedback
you think you're done
then nothing happens
sman-81 closed pull request #544:
URL: https://github.com/apache/logging-log4j2/pull/544
--
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: notificat
sman-81 commented on pull request #544:
URL: https://github.com/apache/logging-log4j2/pull/544#issuecomment-1028224532
"we love pull requests" they say
you send them prs
you listen to their criticism
you incorporate their feedback
you think you're done
then nothing happens
avandeursen commented on pull request #720:
URL: https://github.com/apache/logging-log4j2/pull/720#issuecomment-1028227148
This PR seems a bit stalled, so let me re-explain why I think it's important
to merge it:
- Doing a lookup in structured data with a null key is valid according
avandeursen edited a comment on pull request #720:
URL: https://github.com/apache/logging-log4j2/pull/720#issuecomment-1028227148
This PR seems a bit stalled. I understand everyone is super busy, so let me
briefly explain why I think it's important to merge it:
- Doing a lookup in s
jvz commented on pull request #544:
URL: https://github.com/apache/logging-log4j2/pull/544#issuecomment-1028315065
PRs are welcome; the PMC are all volunteers, though, who don't work on this
project full time. Before each release, we usually go over open PRs and Jira
issues to make sure to
carterkozak commented on pull request #542:
URL: https://github.com/apache/logging-log4j2/pull/542#issuecomment-1028513533
@sman-81 we are all volunteers and have limited time to devote to review.
Your most recent comment comes across as entitled and rude. Please work on
that. In your posi
dependabot[bot] opened a new pull request #740:
URL: https://github.com/apache/logging-log4j2/pull/740
Bumps
[elasticsearch-rest-high-level-client](https://github.com/elastic/elasticsearch)
from 7.16.3 to 7.17.0.
Release notes
Sourced from https://github.com/elastic/elasticsearch
sman-81 commented on pull request #542:
URL: https://github.com/apache/logging-log4j2/pull/542#issuecomment-1028625847
@carterkozak
I'm a volunteer just like you.
My pull request is from July 1st, thus seven months old by now!
The last comment from anyone other than me was Augus
vy merged pull request #740:
URL: https://github.com/apache/logging-log4j2/pull/740
--
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-
carterkozak commented on pull request #542:
URL: https://github.com/apache/logging-log4j2/pull/542#issuecomment-1029134939
The point is that `string.split` is inefficient, regardless of internal
details because it's allocating several new strings unnecessarily.
--
This is an automated me
carterkozak commented on a change in pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#discussion_r798723570
##
File path:
log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/PropertiesLookup.java
##
@@ -29,29 +32,28 @@
*/
public final clas
carterkozak commented on pull request #732:
URL: https://github.com/apache/logging-log4j2/pull/732#issuecomment-1029409723
I’m having a bit of an internal struggle with this — in its current state,
it changes the behavior of `$${ctx:foo}` used in properties because today we
build propertie
dependabot[bot] opened a new pull request #741:
URL: https://github.com/apache/logging-log4j2/pull/741
Bumps `slf4jVersion` from 1.7.25 to 1.7.35.
Updates `slf4j-api` from 1.7.25 to 1.7.35
Commits
https://github.com/qos-ch/slf4j/commit/02860b67ef7ff39fa9c7d98fd00da2ee913faed
vy merged pull request #741:
URL: https://github.com/apache/logging-log4j2/pull/741
--
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-
vy merged pull request #661:
URL: https://github.com/apache/logging-log4j2/pull/661
--
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-
vy merged pull request #610:
URL: https://github.com/apache/logging-log4j2/pull/610
--
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-
4701 - 4800 of 7264 matches
Mail list logo
