Apply suggestions from code review

Co-authored-by: Thomas Labarussias <[email protected]> Signed-off-by: Carlo Mencarelli <[email protected]>
falcosecurity · Oct 10, 2023 · 85d2573 · 85d2573
1 parent e65809c
commit 85d2573
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/config.go b/config.go
@@ -159,6 +159,7 @@ func getConfig() *types.Configuration {
 	v.SetDefault("SumoLogic.SourceHost", "")
 	v.SetDefault("SumoLogic.Name", "")
 	v.SetDefault("SumoLogic.CheckCert", true)
+	v.SetDefault("SumoLogic.MutualTLS", false)
 
 	v.SetDefault("AWS.AccessKeyID", "")
 	v.SetDefault("AWS.SecretAccessKey", "")

diff --git a/config_example.yaml b/config_example.yaml
@@ -485,9 +485,9 @@ dynatrace:
   # checkcert: true # check if ssl certificate of the output is valid (default: true)
 
 sumologic:
-  ReceiverURL: "" # Sumologic HTTP Source URL, if not empty, Sumologic output is enabled
-  # SourceCategory: "" # Override the default Sumologic Source Category
-  # SourceHost: "" # Override the default Sumologic Source Host
-  # Name: "" # Override the default Sumologic Source Name
+  receiverURL: "" # Sumologic HTTP Source URL, if not empty, Sumologic output is enabled
+  # sourceCategory: "" # Override the default Sumologic Source Category
+  # sourceHost: "" # Override the default Sumologic Source Host
+  # name: "" # Override the default Sumologic Source Name
   # minimumpriority: "" # minimum priority of event for using this output, order is emergency|alert|critical|error|warning|notice|informational|debug or "" (default)
   # checkcert: true # check if ssl certificate of the output is valid (default: true)
diff --git a/docs/outputs/sumologic.md b/docs/outputs/sumologic.md
@@ -31,9 +31,9 @@ The Env var values override the settings from yaml file.
 
 ```yaml
 sumologic:
-  ReceiverURL: "" # Sumologic HTTP Source URL, if not empty, Sumologic output is enabled
-  # SourceCategory: "" # Override the default Sumologic Source Category
-  # SourceHost: "" # Override the default Sumologic Source Host
+  receiverURL: "" # Sumologic HTTP Source URL, if not empty, Sumologic output is enabled
+  # sourceCategory: "" # Override the default Sumologic Source Category
+  # sourceHost: "" # Override the default Sumologic Source Host
   # Name: "" # Override the default Sumologic Source Name
   # minimumpriority: "" # minimum priority of event for using this output, order is emergency|alert|critical|error|warning|notice|informational|debug or "" (default)
   # checkcert: true # check if ssl certificate of the output is valid (default: true)

diff --git a/outputs/sumologic.go b/outputs/sumologic.go
@@ -35,7 +35,7 @@ func (c *Client) SumoLogicPost(falcopayload types.FalcoPayload) {
 	err = c.Post(falcopayload)
 	if err != nil {
 		c.setSumoLogicErrorMetrics()
-		log.Printf("[ERROR] : SumoLogic - %v\n", err)
+		log.Printf("[ERROR] : %x - %v\n", c.OutputType, err)
 		return
 	}