Updating logging property and update string manipulation (#208)

* Updating logging property and update string manipulation

* Address PR comment

---------

Co-authored-by: elvilla <elvilla@microsoft.com>

src/common.tests/Utilities/StringManipulationTests.cs

@@ -0,0 +1,24 @@
+// --------------------------------------------------------------------------------------------
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+// --------------------------------------------------------------------------------------------
+
+using Microsoft.BridgeToKubernetes.Common.Utilities;
+using Xunit;
+
+namespace Microsoft.BridgeToKubernetes.Common.Tests.Utilities
+{
+    public class StringManipulationTests
+    {
+        [Theory]
+        [InlineData("BEGIN PRIVATE KEY\nuirethweuifhewiufhweiuofhweo\njweiorjweoifnewouhfousdgherufhsdiuhEND PRIVATE KEY\n", "KEY_WAS_REMOVED\n")]
+        [InlineData("iurhuweifhoBEGIN PRIVATE KEY\nuirethweuifhewiufhweiuofhweo\njweiorjweoifnewouhfousdgherufhsdiuhEND PRIVATE KEY\n", "iurhuweifhoKEY_WAS_REMOVED\n")]
+        [InlineData("BEGIN PRIVATE KEYb2kkkkkkkkkkkkkkEND PRIVATE KEY", "KEY_WAS_REMOVED")]
+        [InlineData("hello world", "hello world")]
+        [InlineData("BEGIN PRIVATE KEY\nuirethweuifhewiufhweiuofhweo\njweiorjweoifnewouhfousdgherufhsdiuhEND PRIVATE KEY", "KEY_WAS_REMOVED")]
+        public void RemovePrivateKeyIfNeededTest(string input, string expected)
+        {
+            Assert.True(string.Equals(StringManipulation.RemovePrivateKeyIfNeeded(input), expected));
+        }
+    }
+}

src/common/Utilities/StringManipulation.cs

@@ -15,7 +15,7 @@ public static string RemovePrivateKeyIfNeeded(string inputString)
         {
             if (inputString.ContainsIgnoreCase("BEGIN PRIVATE KEY"))
             {
-                return Regex.Replace(inputString, "BEGIN PRIVATE KEY.*END PRIVATE KEY", "KEY_WAS_REMOVED");
+                return Regex.Replace(inputString, @"(BEGIN PRIVATE KEY(\s|.)*END PRIVATE KEY)", "KEY_WAS_REMOVED");
             }
             return inputString;
         }

src/dsc/Telemetry/CommandLogging.cs

@@ -54,8 +54,7 @@ public void Finished(bool success, string failureReason)
             if (!string.IsNullOrWhiteSpace(failureReason))
             {
                 // Ensure no secrets are logged
-
-                properties.Add("FailureReason", StringManipulation.RemovePrivateKeyIfNeeded(failureReason));
+                properties.Add("FailureReason2", StringManipulation.RemovePrivateKeyIfNeeded(failureReason));
             }
 
             this._log?.Event(