Return-Path: X-Original-To: apmail-cloudstack-commits-archive@www.apache.org Delivered-To: apmail-cloudstack-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 91C4710347 for ; Tue, 8 Apr 2014 09:01:18 +0000 (UTC) Received: (qmail 79851 invoked by uid 500); 8 Apr 2014 09:01:18 -0000 Delivered-To: apmail-cloudstack-commits-archive@cloudstack.apache.org Received: (qmail 79598 invoked by uid 500); 8 Apr 2014 09:01:17 -0000 Mailing-List: contact commits-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list commits@cloudstack.apache.org Received: (qmail 79577 invoked by uid 99); 8 Apr 2014 09:01:14 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 08 Apr 2014 09:01:14 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id A5CE494DD21; Tue, 8 Apr 2014 09:01:13 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: devdeep@apache.org To: commits@cloudstack.apache.org Date: Tue, 08 Apr 2014 09:01:14 -0000 Message-Id: <7595f9e667c94b54bfd541b98d69e01f@git.apache.org> In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [2/2] git commit: updated refs/heads/4.4 to 9bc19bc CLOUDSTACK-6326: fixed password visible in plain text in some of commands in Hyper-v Agent logs Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/9bc19bc2 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/9bc19bc2 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/9bc19bc2 Branch: refs/heads/4.4 Commit: 9bc19bc241a55f92bc423681b3457b4decca9863 Parents: 3f84cc6 Author: Anshul Gangwar Authored: Wed Apr 2 02:13:28 2014 -0700 Committer: Devdeep Singh Committed: Tue Apr 8 14:17:38 2014 +0530 ---------------------------------------------------------------------- .../HypervResource/HypervResourceController.cs | 58 ++++++++++---------- .../ServerResource/HypervResource/Utils.cs | 4 +- 2 files changed, 31 insertions(+), 31 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9bc19bc2/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs index 66b9828..4f4c14e 100644 --- a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs +++ b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs @@ -173,7 +173,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.SetupCommand + cmd.ToString()); + logger.Info(CloudStackTypes.SetupCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -323,7 +323,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.RebootCommand + cmd.ToString()); + logger.Info(CloudStackTypes.RebootCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -367,7 +367,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.DestroyCommand + cmd.ToString()); + logger.Info(CloudStackTypes.DestroyCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -375,7 +375,7 @@ namespace HypervResource try { // Assert - String errMsg = "No 'volume' details in " + CloudStackTypes.DestroyCommand + " " + cmd.ToString(); + String errMsg = "No 'volume' details in " + CloudStackTypes.DestroyCommand + " " + Utils.CleanString(cmd.ToString()); if (cmd.volume == null) { logger.Error(errMsg); @@ -430,7 +430,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.DestroyCommand + cmd.ToString()); + logger.Info(CloudStackTypes.DestroyCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -438,7 +438,7 @@ namespace HypervResource try { // Assert - String errMsg = "No 'volume' details in " + CloudStackTypes.DestroyCommand + " " + cmd.ToString(); + String errMsg = "No 'volume' details in " + CloudStackTypes.DestroyCommand + " " + Utils.CleanString(cmd.ToString()); VolumeObjectTO destVolumeObjectTO = VolumeObjectTO.ParseJson(cmd.data); if (destVolumeObjectTO.name == null) @@ -756,7 +756,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CheckHealthCommand + cmd.ToString()); + logger.Info(CloudStackTypes.CheckHealthCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -774,7 +774,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CheckOnHostCommand + cmd.ToString()); + logger.Info(CloudStackTypes.CheckOnHostCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -793,7 +793,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CheckSshCommand + cmd.ToString()); + logger.Info(CloudStackTypes.CheckSshCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -811,7 +811,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CheckVirtualMachineCommand + cmd.ToString()); + logger.Info(CloudStackTypes.CheckVirtualMachineCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; string vmName = cmd.vmName; @@ -848,7 +848,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.DeleteStoragePoolCommand + cmd.ToString()); + logger.Info(CloudStackTypes.DeleteStoragePoolCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -991,7 +991,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.PlugNicCommand + cmd.ToString()); + logger.Info(CloudStackTypes.PlugNicCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { @@ -1011,7 +1011,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CleanupNetworkRulesCmd + cmd.ToString()); + logger.Info(CloudStackTypes.CleanupNetworkRulesCmd + Utils.CleanString(cmd.ToString())); object ansContent = new { result = false, @@ -1029,7 +1029,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.CheckNetworkCommand + cmd.ToString()); + logger.Info(CloudStackTypes.CheckNetworkCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -1047,7 +1047,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.ReadyCommand + cmd.ToString()); + logger.Info(CloudStackTypes.ReadyCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { result = true, @@ -1132,7 +1132,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.StopCommand + cmd.ToString()); + logger.Info(CloudStackTypes.StopCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -1228,7 +1228,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.MaintainCommand + cmd.ToString()); + logger.Info(CloudStackTypes.MaintainCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { @@ -1251,7 +1251,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.PingRoutingCommand + cmd.ToString()); + logger.Info(CloudStackTypes.PingRoutingCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { @@ -1273,7 +1273,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.PingCommand + cmd.ToString()); + logger.Info(CloudStackTypes.PingCommand + Utils.CleanString(cmd.ToString())); object ansContent = new { @@ -1295,7 +1295,7 @@ namespace HypervResource using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.ModifyVmNicConfigCommand + cmd.ToString()); + logger.Info(CloudStackTypes.ModifyVmNicConfigCommand + Utils.CleanString(cmd.ToString())); bool result = false; String vmName = cmd.vmName; uint vlan = (uint)cmd.vlan; @@ -1329,7 +1329,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.GetVmConfigCommand + cmd.ToString()); + logger.Info(CloudStackTypes.GetVmConfigCommand + Utils.CleanString(cmd.ToString())); bool result = false; String vmName = cmd.vmName; ComputerSystem vm = wmiCallsV2.GetComputerSystem(vmName); @@ -1383,7 +1383,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.GetVmStatsCommand + cmd.ToString()); + logger.Info(CloudStackTypes.GetVmStatsCommand + Utils.CleanString(cmd.ToString())); bool result = false; JArray vmNamesJson = cmd.vmNames; string[] vmNames = vmNamesJson.ToObject(); @@ -1894,7 +1894,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.GetHostStatsCommand + cmd.ToString()); + logger.Info(CloudStackTypes.GetHostStatsCommand + Utils.CleanString(cmd.ToString())); bool result = false; string details = null; object hostStats = null; @@ -1956,7 +1956,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.PrepareForMigrationCommand + cmd.ToString()); + logger.Info(CloudStackTypes.PrepareForMigrationCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = true; @@ -1990,7 +1990,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.MigrateCommand + cmd.ToString()); + logger.Info(CloudStackTypes.MigrateCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -2026,7 +2026,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.MigrateVolumeCommand + cmd.ToString()); + logger.Info(CloudStackTypes.MigrateVolumeCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -2063,7 +2063,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.MigrateWithStorageCommand + cmd.ToString()); + logger.Info(CloudStackTypes.MigrateWithStorageCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -2249,7 +2249,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.GetVncPortCommand + cmd.ToString()); + logger.Info(CloudStackTypes.GetVncPortCommand + Utils.CleanString(cmd.ToString())); string details = null; bool result = false; @@ -2288,7 +2288,7 @@ namespace HypervResource { using (log4net.NDC.Push(Guid.NewGuid().ToString())) { - logger.Info(CloudStackTypes.HostVmStateReportCommand + cmd.ToString()); + logger.Info(CloudStackTypes.HostVmStateReportCommand + Utils.CleanString(cmd.ToString())); string details = null; Dictionary[] hostVmStateReport = null; http://git-wip-us.apache.org/repos/asf/cloudstack/blob/9bc19bc2/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs index 6ebc5bf..5ddcbef 100644 --- a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs +++ b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs @@ -167,8 +167,8 @@ namespace HypervResource public static string CleanString(string stringToClean) { string cleanString = null; - string regexQueryString = "(&|%26)?(password|accesskey|secretkey)(=|%3D).*?(?=(%26|[&'\"]))"; - string regexJson = "\"(password|accesskey|secretkey)\":\".*?\",?"; + string regexQueryString = "(&|%26)?(password|accesskey|secretkey|Password)(=|%3D).*?(?=(%26|[&'\"]))"; + string regexJson = "\"(password|accesskey|secretkey|Password)\":\\s?\".*?\",?"; cleanString = System.Text.RegularExpressions.Regex.Replace(stringToClean, regexQueryString, ""); cleanString = System.Text.RegularExpressions.Regex.Replace(cleanString, regexJson, ""); return cleanString;