Add LogOnlyErrors flag to each config section (default true). Messages succesfully posted to Mattermost will not be logged to the console, otherwise it gets too busy

Author: Adam Warner

Matterhook.NET/Config.cs

@@ -11,10 +11,12 @@ public class Config
 
     public class DiscourseConfig
     {
+        public bool LogOnlyErrors { get; set; } = true;
         public string Secret { get; set; }
         public string[] IgnoredTopicTitles { get; set; }
         public bool IgnorePrivateMessages { get; set; }
         public MattermostConfig MattermostConfig { get; set; }
+        
     }
 
     public class MattermostConfig
@@ -28,6 +30,7 @@ public class MattermostConfig
     //TODO: Look at configuring subscribed events
     public class GithubConfig
     {
+        public bool LogOnlyErrors { get; set; } = true;
         public string Secret { get; set; }
         public MattermostConfig DefaultMattermostConfig { get; set; }
         public List<RepoConfig> RepoList { get; set; }
@@ -42,6 +45,7 @@ public class RepoConfig
 
     public class DockerHubConfig
     {
+        public bool LogOnlyErrors { get; set; } = true;
         public MattermostConfig DefaultMattermostConfig { get; set; }
         public List<RepoConfig> RepoList { get; set; }
     }

Matterhook.NET/Controllers/DiscourseHookController.cs

@@ -97,14 +97,18 @@ public async Task<IActionResult> Receive()
                             ? $"Unable to post to Mattermost {response.StatusCode}"
                             : "Unable to post to Mattermost");
 
-                        return StatusCode(500,response != null
-                            ? $"Problem posting to Mattermost: {response.StatusCode}"
-                            : "Problem Posting to Mattermost");
+                        return StatusCode(500, response != null
+                            ? $"Unable to post to Mattermost: {response.StatusCode}"
+                            : "Unable to post to Mattermost");
                     }
 
-                    if (message != null) stuffToLog.Add(message.Text);
-                    stuffToLog.Add("Succesfully posted to Mattermost");
-                    Util.LogList(stuffToLog);
+                    if (!_config.LogOnlyErrors)
+                    {
+                        if (message != null) stuffToLog.Add(message.Text);
+                        stuffToLog.Add("Succesfully posted to Mattermost");
+                        Util.LogList(stuffToLog);
+                    }
+                    
                     return StatusCode(200, "Succesfully posted to Mattermost");
                 }
 

Matterhook.NET/Controllers/DockerHubHookController.cs

@@ -79,13 +79,17 @@ public async Task<IActionResult> Receive()
                         : "Unable to post to Mattermost");
 
                     return StatusCode(500, response != null
-                        ? $"Problem posting to Mattermost: {response.StatusCode}"
-                        : "Problem Posting to Mattermost");
+                        ? $"Unable to post to Mattermost: {response.StatusCode}"
+                        : "Unable to post to Mattermost");
                 }
 
-                stuffToLog.Add(msg.Text);
-                stuffToLog.Add("Succesfully posted to Mattermost");
-                Util.LogList(stuffToLog);
+                if (!_config.LogOnlyErrors)
+                {
+                    stuffToLog.Add(msg.Text);
+                    stuffToLog.Add("Succesfully posted to Mattermost");
+                    Util.LogList(stuffToLog);
+                }
+                
                 return StatusCode(200, "Succesfully posted to Mattermost");
             }
             catch (Exception e)

Matterhook.NET/Controllers/GitHubHookController.cs

@@ -144,12 +144,17 @@ public async Task<IActionResult> Receive()
                             : "Unable to post to Mattermost");
 
                         return StatusCode(500, response != null
-                            ? $"Problem posting to Mattermost: {response.StatusCode}"
-                            : "Problem Posting to Mattermost");
+                            ? $"Unable to post to Mattermost: {response.StatusCode}"
+                            : "Unable to post to Mattermost");
                     }
-                    if (message != null) stuffToLog.Add(message.Text);
-                    stuffToLog.Add("Succesfully posted to Mattermost");
-                    Util.LogList(stuffToLog);
+
+                    if (!_config.LogOnlyErrors)
+                    {
+                        if (message != null) stuffToLog.Add(message.Text);
+                        stuffToLog.Add("Succesfully posted to Mattermost");
+                        Util.LogList(stuffToLog);
+                    }
+
                     return StatusCode(200, "Succesfully posted to Mattermost");
                 }
                 stuffToLog.Add("Invalid Signature!");