[LOG4J2-623] Better structure of Thread Context Map in JSONLayout

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0-rc1, 2.0, 2.0.1
• Fix Version/s: 2.0-rc2
• Component/s: Layouts
• Assignee: Remko Popma
• Reporter: Mikael Ståldal
• Created: 29/Apr/14 15:12
• Updated: 20/Jul/16 16:42
• Resolved: 24/May/16 16:22

Description

Currently, the Thread Context Map looks like this in JSONLayout:

Current

 "Properties":[
      {
        "name":"UserName",
        "value":"admin"
      },
      {
        "name":"OrgName",
        "value":"test"
      } 
    ] 

This does not properly make use of the JSON data format. Since the Thread Context Map is a map, it should be represented as a JSON object. And why not name it "mdc" rather than the quite vauge "Properties"?

Suggested

  "mdc": {
    "UserName":"admin", 
    "OrgName":"test"
  }

Issue Links

• Blocked: LOG4J2-1469 I wish the json layout used a map structure for the context map instead of a list of map entry objects.
• relates to: LOG4J2-1362 Create a YAML layout