Made MessageTemplate capitalized to match Serilog defaults

redboxllc · Jun 15, 2020 · 372a622 · 372a622
1 parent 0ea08c5
commit 372a622
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -59,7 +59,7 @@ Stackdriver will break the long line into multiple lines, which will break searc
 
 #### includeMessageTemplate
 
-Default `true`.  If the Serilog Message Template should be included in the logs, e.g. ` { ... "messageTemplate" : "Hello from {name:l}" ... }`
+Default `true`.  If the Serilog Message Template should be included in the logs, e.g. ` { ... "MessageTemplate" : "Hello from {name:l}" ... }`
 
 #### valueFormatter
 

diff --git a/src/Redbox.Serilog.Stackdriver.Tests/StackdriverFormatterTests.cs b/src/Redbox.Serilog.Stackdriver.Tests/StackdriverFormatterTests.cs
@@ -91,6 +91,9 @@ private void AssertValidLogLine(Dictionary<string, string> logDict,
 
             Assert.True(logDict.ContainsKey("severity"));
             Assert.NotEmpty(logDict["severity"]);
+
+            Assert.True(logDict.ContainsKey(("MessageTemplate")));
+            Assert.NotEmpty(logDict["MessageTemplate"]);
 
             if (hasException)
             {

diff --git a/src/Redbox.Serilog.Stackdriver/StackdriverJsonFormatter.cs b/src/Redbox.Serilog.Stackdriver/StackdriverJsonFormatter.cs
@@ -114,7 +114,8 @@ public void FormatEvent(LogEvent logEvent, TextWriter originalOutput, JsonValueF
             // Serilog Message Template
             if (_includeMessageTemplate)
             {
-                output.Write(",\"messageTemplate\":");
+                // Capitalized to match default Serilog JsonFormatter
+                output.Write(",\"MessageTemplate\":");
                 JsonValueFormatter.WriteQuotedJsonString(logEvent.MessageTemplate.Text, output);
             }