You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@logging.apache.org by jhovell <gi...@git.apache.org> on 2018/02/04 19:14:30 UTC

[GitHub] logging-log4j2 pull request #146: fixes capitalization of log4j2.ContextData...

GitHub user jhovell opened a pull request:

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

    fixes capitalization of log4j2.ContextDataInjector property name

    See here:
    
    https://github.com/apache/logging-log4j2/blob/master/log4j-core/src/main/java/org/apache/logging/log4j/core/impl/ContextDataInjectorFactory.java#L65
    


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

    $ git pull https://github.com/jhovell/logging-log4j2 fix-ContextDataInjector-documentation

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

    https://github.com/apache/logging-log4j2/pull/146.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 #146
    
----
commit 819b6d458467bc787ccc17b22704fe63ee58d8f0
Author: John Hovell <jo...@...>
Date:   2018-02-04T19:13:51Z

    fixes capitalization of log4j2.ContextDataInjector property name

----


---

[GitHub] logging-log4j2 issue #146: fixes capitalization of log4j2.ContextDataInjecto...

Posted by jvz <gi...@git.apache.org>.

Github user jvz commented on the issue:

    https://github.com/apache/logging-log4j2/pull/146
  
    It's supposed to be lower case now with the new property naming scheme in use. It would be more useful to update existing code/docs to use the normalized property names instead IMO.


---