[GitHub] [logging-log4cxx] ams-tschoening commented on a change in pull request #95: Add short filename to location info

2022-01-03 Thread GitBox
ams-tschoening commented on a change in pull request #95: URL: https://github.com/apache/logging-log4cxx/pull/95#discussion_r777873523 ## File path: src/main/cpp/locationinfo.cpp ## @@ -45,16 +45,35 @@ LocationInfo::LocationInfo( const char* const fileName1, int lineNum

[jira] [Commented] (LOG4J2-3298) Update JSONTemplateFormat to support not escaping certain payloads

2022-01-03 Thread Ralph Goers (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468374#comment-17468374 ] Ralph Goers commented on LOG4J2-3298: - This only makes sense when the referenced dat

[GitHub] [logging-log4cxx] rm5248 merged pull request #98: Save abi compliance check

2022-01-03 Thread GitBox
rm5248 merged pull request #98: URL: https://github.com/apache/logging-log4cxx/pull/98 -- 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] [logging-log4cxx] swebb2066 commented on pull request #96: Reduce indirection in Logger and Hiearchy implementations

2022-01-03 Thread GitBox
swebb2066 commented on pull request #96: URL: https://github.com/apache/logging-log4cxx/pull/96#issuecomment-1004441759 To copy a shared_ptr the reference count must be incremented, for which a lock must be acquired. By returning a const reference, the caller can decide whether to co

[GitHub] [logging-log4cxx] rm5248 merged pull request #97: Allow throughputtests to be built when LOGCHAR_IS_WCHAR is on

2022-01-03 Thread GitBox
rm5248 merged pull request #97: URL: https://github.com/apache/logging-log4cxx/pull/97 -- 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] [logging-log4cxx] rm5248 merged pull request #96: Reduce indirection in Logger and Hiearchy implementations

2022-01-03 Thread GitBox
rm5248 merged pull request #96: URL: https://github.com/apache/logging-log4cxx/pull/96 -- 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] [logging-log4cxx] rm5248 commented on pull request #96: Reduce indirection in Logger and Hiearchy implementations

2022-01-03 Thread GitBox
rm5248 commented on pull request #96: URL: https://github.com/apache/logging-log4cxx/pull/96#issuecomment-1004427963 Looks good to me. The ABI failure is not due to this change, but due to some of the other changes that you've made(e.g. returning a reference instead of the `shared_p

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #95: Add short filename to location info

2022-01-03 Thread GitBox
rm5248 commented on a change in pull request #95: URL: https://github.com/apache/logging-log4cxx/pull/95#discussion_r44337 ## File path: src/main/include/log4cxx/spi/location/locationinfo.h ## @@ -132,9 +152,28 @@ class LOG4CXX_EXPORT LocationInfo #if !defined(__LOG4CXX_FU

[jira] [Closed] (LOG4J2-3306) OptionConverter can get a StackOverflowError.

2022-01-03 Thread Ralph Goers (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3306?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ralph Goers closed LOG4J2-3306. --- Resolution: Fixed > OptionConverter can get a StackOverflowError. >

[GitHub] [logging-log4cxx] rm5248 commented on a change in pull request #95: Add short filename to location info

2022-01-03 Thread GitBox
rm5248 commented on a change in pull request #95: URL: https://github.com/apache/logging-log4cxx/pull/95#discussion_r36412 ## File path: src/main/cpp/locationinfo.cpp ## @@ -45,16 +45,35 @@ LocationInfo::LocationInfo( const char* const fileName1, int lineNumber1 )

[jira] [Commented] (LOG4J2-3306) OptionConverter can get a StackOverflowError.

2022-01-03 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468241#comment-17468241 ] ASF subversion and git services commented on LOG4J2-3306: - Commi

[jira] [Commented] (LOG4J2-3306) OptionConverter can get a StackOverflowError.

2022-01-03 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468240#comment-17468240 ] ASF subversion and git services commented on LOG4J2-3306: - Commi

[GitHub] [logging-log4cxx] rm5248 commented on pull request #86: Update log4cxx.h.in

2022-01-03 Thread GitBox
rm5248 commented on pull request #86: URL: https://github.com/apache/logging-log4cxx/pull/86#issuecomment-1004401456 > The utility of LOG4CXX_VERSION_GET_{MAJOR,MINOR,PATCH,TWEAK} would be more obvious if we were to add a version identifier object (i.e. an externally visible integer value)

[jira] [Updated] (LOG4J2-2903) Cannot log to HDFS with a custom appender?

2022-01-03 Thread Matt Sicker (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-2903?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Sicker updated LOG4J2-2903: Fix Version/s: (was: Scala 12.0) > Cannot log to HDFS with a custom appender? > --

[jira] [Updated] (LOG4J2-3184) Support Scala 3

2022-01-03 Thread Matt Sicker (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Sicker updated LOG4J2-3184: Fix Version/s: Scala 12.1 (was: Scala 12.0) > Support Scala 3 > ---

[jira] [Updated] (LOG4J2-3184) Support Scala 3

2022-01-03 Thread Matt Sicker (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Sicker updated LOG4J2-3184: Fix Version/s: Scala 12.0 (was: Scala 13.0) > Support Scala 3 > ---

[GitHub] [logging-log4j-scala] jvz merged pull request #11: log4j 2.17.1

2022-01-03 Thread GitBox
jvz merged pull request #11: URL: https://github.com/apache/logging-log4j-scala/pull/11 -- 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: notificati

[GitHub] [logging-log4j-scala] jvz merged pull request #12: sbt 1.6.1

2022-01-03 Thread GitBox
jvz merged pull request #12: URL: https://github.com/apache/logging-log4j-scala/pull/12 -- 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: notificati

[jira] [Updated] (LOG4J2-3298) Update JSONTemplateFormat to support not escaping certain payloads

2022-01-03 Thread Matt Pavlovich (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Pavlovich updated LOG4J2-3298: --- Description: Currently, if a JSON string is pushed to ThreadContext, it will be escaped. It

[jira] [Updated] (LOG4J2-3298) Update JSONTemplateFormat to support not escaping certain payloads

2022-01-03 Thread Matt Pavlovich (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Pavlovich updated LOG4J2-3298: --- Description: Currently, if a JSON string is pushed to ThreadContext, it will be escaped. It

[jira] [Updated] (LOG4J2-3298) Update JSONTemplateFormat to support not escaping certain payloads

2022-01-03 Thread Matt Pavlovich (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matt Pavlovich updated LOG4J2-3298: --- Description: Currently, if a JSON string is pushed to ThreadContext, it will be escaped. It

[jira] [Comment Edited] (LOG4J2-3293) JDBC Appender should use JNDI Manager and JNDI access should be limited.

2022-01-03 Thread Ralph Goers (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468135#comment-17468135 ] Ralph Goers edited comment on LOG4J2-3293 at 1/3/22, 6:15 PM:

[jira] [Commented] (LOG4J2-3293) JDBC Appender should use JNDI Manager and JNDI access should be limited.

2022-01-03 Thread Ralph Goers (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468135#comment-17468135 ] Ralph Goers commented on LOG4J2-3293: - There are two cases: # Someone can modify yo

[jira] [Commented] (LOG4J2-3281) PropertiesConfiguration.buildAppender not adding filters to custom appender

2022-01-03 Thread Gary D. Gregory (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468103#comment-17468103 ] Gary D. Gregory commented on LOG4J2-3281: - [~coredumped] Well, crud, I am so sor

[jira] [Comment Edited] (LOG4J2-3293) JDBC Appender should use JNDI Manager and JNDI access should be limited.

2022-01-03 Thread Eric Everman (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468049#comment-17468049 ] Eric Everman edited comment on LOG4J2-3293 at 1/3/22, 3:05 PM: ---

[jira] [Commented] (LOG4J2-3293) JDBC Appender should use JNDI Manager and JNDI access should be limited.

2022-01-03 Thread Eric Everman (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468049#comment-17468049 ] Eric Everman commented on LOG4J2-3293: -- Could someone please clarify this detail in

[jira] [Commented] (LOG4J2-3281) PropertiesConfiguration.buildAppender not adding filters to custom appender

2022-01-03 Thread Jira
[ https://issues.apache.org/jira/browse/LOG4J2-3281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468038#comment-17468038 ] Fábio Constantino commented on LOG4J2-3281: --- [~ggregory]  Just double checked

[jira] [Commented] (LOG4J2-3281) PropertiesConfiguration.buildAppender not adding filters to custom appender

2022-01-03 Thread Gary D. Gregory (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17468005#comment-17468005 ] Gary D. Gregory commented on LOG4J2-3281: - If you use Maven to build make sure t

[jira] [Assigned] (LOG4J2-3298) Update JSONTemplateFormat to support not escaping certain payloads

2022-01-03 Thread Volkan Yazici (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Volkan Yazici reassigned LOG4J2-3298: - Assignee: Volkan Yazici > Update JSONTemplateFormat to support not escaping certain pay

[jira] [Assigned] (LOG4J2-3303) Add support for FileSize in TB

2022-01-03 Thread Volkan Yazici (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Volkan Yazici reassigned LOG4J2-3303: - Assignee: Volkan Yazici > Add support for FileSize in TB >

[jira] [Commented] (LOG4J2-3302) Xml configuration of ClassArbiter does not work

2022-01-03 Thread Volkan Yazici (Jira)
[ https://issues.apache.org/jira/browse/LOG4J2-3302?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17467962#comment-17467962 ] Volkan Yazici commented on LOG4J2-3302: --- Thanks so much for the report [~nielsull]

[jira] [Created] (LOG4J2-3308) Replace JsonTemplateLayout test-JAR with a separate module

2022-01-03 Thread Volkan Yazici (Jira)
Volkan Yazici created LOG4J2-3308: - Summary: Replace JsonTemplateLayout test-JAR with a separate module Key: LOG4J2-3308 URL: https://issues.apache.org/jira/browse/LOG4J2-3308 Project: Log4j 2

[jira] [Comment Edited] (LOG4J2-3281) PropertiesConfiguration.buildAppender not adding filters to custom appender

2022-01-03 Thread Jira
[ https://issues.apache.org/jira/browse/LOG4J2-3281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17467926#comment-17467926 ] Fábio Constantino edited comment on LOG4J2-3281 at 1/3/22, 10:54 AM: -

[jira] [Commented] (LOG4J2-3281) PropertiesConfiguration.buildAppender not adding filters to custom appender

2022-01-03 Thread Jira
[ https://issues.apache.org/jira/browse/LOG4J2-3281?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17467926#comment-17467926 ] Fábio Constantino commented on LOG4J2-3281: --- [~ggregory]  Just tested working

[GitHub] [logging-log4cxx] ams-tschoening commented on a change in pull request #95: Add short filename to location info

2022-01-03 Thread GitBox
ams-tschoening commented on a change in pull request #95: URL: https://github.com/apache/logging-log4cxx/pull/95#discussion_r777344076 ## File path: src/main/cpp/locationinfo.cpp ## @@ -45,16 +45,35 @@ LocationInfo::LocationInfo( const char* const fileName1, int lineNum