Uploaded image for project: 'SLF4J'
  1. SLF4J
  2. SLF4J-422

Have slf4j-log4j12 include extra arguments to Logger methods

    XMLWordPrintable

Details

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Major Major
    • None
    • 1.7.22
    • Implementations
    • None
    • All

    • enhancement

    Description

      The purpose of logging to slf4j is to allow code to log information without worrying about which implementation and format will be used for actual logging.  In deployed environments, we prefer JSON logging using logstash to be consumed by Splunk, which handles JSON very nicely.  However, for local development and testing, we prefer text output via log4j or logback-classic.  

      We often want to log extra fields to show up in the JSON output such as 

      {{{ "timestamp": "2017...", "message": "Event X happened", "userName": "bob", "url": "http://..."}}}

      To accomplish this, we use logstash's StructuredArgument to provide extra arguments to the Logger:

      import static net.logstash.logback.argument.StructuredArguments.keyValue;
      ...
        LOGGER.info("Event X happened", keyValue("userName", "bob"), keyValue("url", "http://..."));

      We don't want to include {}'s in the message or else it would log the username and url both in the message and as separate JSON properties.

      However, when binding to slf4j-log4j12, it is logged without these extra arguments:

      INFO 2017... Event X happened

      It would help immensely to include these extra arguments automatically if present:

      INFO 2017... Event X happened username=bob url=http://... 

      Attachments

        Activity

          People

            slf4j-dev SLF4J developers list
            epabst Eric Pabst
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: