[
https://issues.apache.org/jira/browse/LOGCXX-513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17241138#comment-17241138
]
Robert Middleton commented on LOGCXX-513:
-
It looks like this is a a simple fix,
carterkozak commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532612187
##
File path:
log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/util/JsonWriter.java
##
@@ -572,6 +565,10 @
carterkozak commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532609807
##
File path:
log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/util/JsonWriter.java
##
@@ -493,18 +492,15
carterkozak commented on pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#issuecomment-735789644
> Even though I was the one who started using
ParameterizedMessage.deepToString(), thinking about it again, I am not happy
with the end result. This breaks the self-con
BUILD UNSTABLE
Build URL
https://ci-builds.apache.org/job/Logging/job/log4j/job/master/191/
Project:
master
Date of build:
Mon, 30 Nov 2020 09:00:44 +
Build duration:
1 hr 22 min and counting
JUnit Tests
Name: liquibase.ext.logging.log4j2 Failed:
BUILD SUCCESS
Build URL
https://ci-builds.apache.org/job/Logging/job/log4j/job/release-2.x/183/
Project:
release-2.x
Date of build:
Mon, 30 Nov 2020 08:56:00 +
Build duration:
1 hr 24 min and counting
JUnit Tests
Name: liquibase.ext.logging.log4j
vy commented on pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#issuecomment-735686828
First and foremost, thanks so much @carterkozak for reviewing the changes
and thinking along for improvements. I have really appreciated them.
`ParameterizedMessage.deepTo
vy commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532469418
##
File path:
log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/util/JsonWriter.java
##
@@ -572,6 +565,10 @@ public
vy commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532463858
##
File path:
log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/util/JsonWriter.java
##
@@ -513,18 +509,15 @@ public
vy commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532463534
##
File path:
log4j-layout-template-json/src/main/java/org/apache/logging/log4j/layout/template/json/util/JsonWriter.java
##
@@ -493,18 +492,15 @@ public
vy commented on a change in pull request #444:
URL: https://github.com/apache/logging-log4j2/pull/444#discussion_r532441454
##
File path:
log4j-api/src/main/java/org/apache/logging/log4j/message/ParameterizedMessage.java
##
@@ -302,6 +302,10 @@ public static String deepToStrin
[
https://issues.apache.org/jira/browse/LOG4J2-2967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17240575#comment-17240575
]
ASF subversion and git services commented on LOG4J2-2967:
-
Commi
[
https://issues.apache.org/jira/browse/LOG4J2-2967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17240573#comment-17240573
]
Volkan Yazici commented on LOG4J2-2967:
---
[~diepet], thanks so much for promptly va
[
https://issues.apache.org/jira/browse/LOG4J2-2967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Volkan Yazici reassigned LOG4J2-2967:
-
Assignee: Volkan Yazici
> NullPointerException when used the messageParameter resolver
[
https://issues.apache.org/jira/browse/LOG4J2-2967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17240572#comment-17240572
]
ASF subversion and git services commented on LOG4J2-2967:
-
Commi
vy commented on pull request #443:
URL: https://github.com/apache/logging-log4j2/pull/443#issuecomment-735649229
@carterkozak Great catch with a complete fix! Thanks so much for sparing
time!
This is an automated message fro
vy merged pull request #443:
URL: https://github.com/apache/logging-log4j2/pull/443
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
[
https://issues.apache.org/jira/browse/LOGCXX-513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17240557#comment-17240557
]
Thorsten Schöning commented on LOGCXX-513:
--
{quote}Is there any fix planned on t
18 matches
Mail list logo
