[docs] Avoid duplicate ToString calls in redaction log processor (#5248)

CodeBlanch · web-flow · commit 0a77cc3cced0 · 2024-01-23T16:46:24.000-08:00
diff --git a/docs/logs/redaction/MyRedactionProcessor.cs b/docs/logs/redaction/MyRedactionProcessor.cs
@@ -31,8 +31,8 @@ public KeyValuePair<string, object> this[int index]
             get
             {
                 var item = this.state[index];
-                var entryVal = item.Value;
-                if (entryVal != null && entryVal.ToString() != null && entryVal.ToString().Contains("<secret>"))
+                var entryVal = item.Value?.ToString();
+                if (entryVal != null && entryVal.Contains("<secret>"))
                 {
                     return new KeyValuePair<string, object>(item.Key, "newRedactedValueHere");
                 }

Original file line number	Diff line number	Diff line change
`@@ -31,8 +31,8 @@ public KeyValuePair<string, object> this[int index]`
`31`	`31`	`get`
`32`	`32`	`{`
`33`	`33`	`var item = this.state[index];`
`34`		`- var entryVal = item.Value;`
`35`		`- if (entryVal != null && entryVal.ToString() != null && entryVal.ToString().Contains("<secret>"))`
	`34`	`+ var entryVal = item.Value?.ToString();`
	`35`	`+ if (entryVal != null && entryVal.Contains("<secret>"))`
`36`	`36`	`{`
`37`	`37`	`return new KeyValuePair<string, object>(item.Key, "newRedactedValueHere");`
`38`	`38`	`}`