You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@logging.apache.org by cakofony <gi...@git.apache.org> on 2017/11/09 13:32:35 UTC

[GitHub] logging-log4j2 pull request #127: LOG4J2-2109: Added "log4j.format.msg.noloo...

GitHub user cakofony opened a pull request:

    https://github.com/apache/logging-log4j2/pull/127

    LOG4J2-2109: Added "log4j.format.msg.nolookups" global configuration

    This allows applications to globally disable message pattern
    from looking up replacements without specifying "%m{nolookups}".

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/cakofony/logging-log4j2 LOG4J2-2109

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/logging-log4j2/pull/127.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #127
    
----
commit 4b51310d8df6cf087eede097480f6d28e4f45866
Author: Carter Kozak <c4...@gmail.com>
Date:   2017-11-09T13:32:14Z

    LOG4J2-2109: Added "log4j.format.msg.nolookups" global configuration
    
    This allows applications to globally disable message pattern
    from looking up replacements without specifying "%m{nolookups}".

----


---

[GitHub] logging-log4j2 issue #127: LOG4J2-2109: Added "log4j.format.msg.nolookups" g...

Posted by jvz <gi...@git.apache.org>.
Github user jvz commented on the issue:

    https://github.com/apache/logging-log4j2/pull/127
  
    Could you rename this option to `log4j2.formatMsgNoLookups` or something similar? We have a new naming scheme in 2.10.0+ which follows `log4j2.camelCasePropertyName`.


---

[GitHub] logging-log4j2 issue #127: LOG4J2-2109: Added "log4j.format.msg.nolookups" g...

Posted by cakofony <gi...@git.apache.org>.
Github user cakofony commented on the issue:

    https://github.com/apache/logging-log4j2/pull/127
  
    There wasn't any test coverage for lookups in `MessagePatternConverterTest` so I've added a couple.


---

[GitHub] logging-log4j2 issue #127: LOG4J2-2109: Added "log4j.formatMsgNoLookups" glo...

Posted by cakofony <gi...@git.apache.org>.
Github user cakofony commented on the issue:

    https://github.com/apache/logging-log4j2/pull/127
  
    Sorry about that, updated


---

[GitHub] logging-log4j2 issue #127: LOG4J2-2109: Added "log4j.formatMsgNoLookups" glo...

Posted by mikaelstaldal <gi...@git.apache.org>.
Github user mikaelstaldal commented on the issue:

    https://github.com/apache/logging-log4j2/pull/127
  
    Please update documentation in `configuration.xml.vm`.


---

[GitHub] logging-log4j2 issue #127: LOG4J2-2109: Added "log4j.format.msg.nolookups" g...

Posted by cakofony <gi...@git.apache.org>.
Github user cakofony commented on the issue:

    https://github.com/apache/logging-log4j2/pull/127
  
    Updated, thanks for the info @jvz!


---

[GitHub] logging-log4j2 pull request #127: LOG4J2-2109: Added "log4j.formatMsgNoLooku...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/logging-log4j2/pull/127


---