From d702f9eefe8456e737bb63ecd8abd96c90088d54 Mon Sep 17 00:00:00 2001 From: Oleg Nenashev Date: Sat, 24 Dec 2016 21:56:51 +0100 Subject: [PATCH] Fixes #54. Do not dump WinSW environment variables to the Event log --- src/Core/ServiceWrapper/Main.cs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/Core/ServiceWrapper/Main.cs b/src/Core/ServiceWrapper/Main.cs index b4cdb08..be83d5e 100644 --- a/src/Core/ServiceWrapper/Main.cs +++ b/src/Core/ServiceWrapper/Main.cs @@ -210,10 +210,13 @@ namespace winsw protected override void OnStart(string[] _) { _envs = _descriptor.EnvironmentVariables; + // TODO: Disabled according to security concerns in https://github.com/kohsuke/winsw/issues/54 + // Could be restored, but unlikely it's required in event logs at all + /** foreach (string key in _envs.Keys) { LogEvent("envar " + key + '=' + _envs[key]); - } + }*/ HandleFileCopies();