Use 'nil' instead of empty map in 'catcher' calls system. Improve code readability and reduce memory allocation overhead.

Author: JocLRojas

soc-ai/configurations/config.go

@@ -40,17 +40,17 @@ func (c *GPTConfig) UpdateGPTConfigurations() {
 
 	for {
 		if err := utils.ConnectionChecker(GPT_API_ENDPOINT); err != nil {
-			catcher.Error("Failed to establish internet connection", err, map[string]any{})
+			catcher.Error("Failed to establish internet connection", err, nil)
 		}
 
 		tempModuleConfig, err := client.GetUTMConfig(enum.SOCAI)
 		if err != nil && err.Error() != "" && err.Error() != " " {
-			catcher.Error("Error while getting GPT configuration", err, map[string]any{})
+			catcher.Error("Error while getting GPT configuration", err, nil)
 			time.Sleep(TIME_FOR_GET_CONFIG * time.Second)
 			continue
 		}
 		if tempModuleConfig == nil {
-			catcher.Error("Got nil config from server", nil, map[string]any{})
+			catcher.Error("Got nil config from server", nil, nil)
 			time.Sleep(TIME_FOR_GET_CONFIG * time.Second)
 			continue
 		}

soc-ai/processor/alertProcessor.go

@@ -21,7 +21,7 @@ func (p *Processor) processAlertsInfo() {
 
 		correlation, err := elastic.FindRelatedAlerts(alertInfo)
 		if err != nil {
-			catcher.Error("error finding related alerts", err, map[string]any{})
+			catcher.Error("error finding related alerts", err, nil)
 		}
 
 		details := schema.ConvertFromAlertToAlertDB(alertInfo)

soc-ai/processor/elastic.go

@@ -30,7 +30,7 @@ func (p *Processor) processAlertToElastic() {
 		if gptConfig.ChangeAlertStatus {
 			err = elastic.ChangeAlertStatus(alert.AlertID, configurations.API_ALERT_COMPLETED_STATUS_CODE, alert.GPTClassification+" - "+alert.GPTReasoning)
 			if err != nil {
-				catcher.Error("error while changing alert status in elastic", err, map[string]any{})
+				catcher.Error("error while changing alert status in elastic", err, nil)
 				continue
 			}
 			catcher.Info("alert status changed to COMPLETED in Panel", map[string]any{"alert": alert.AlertID})
@@ -39,7 +39,7 @@ func (p *Processor) processAlertToElastic() {
 		if gptConfig.AutomaticIncidentCreation && alert.GPTClassification == "possible incident" {
 			incidentsDetails, err := elastic.GetIncidentsByPattern("Incident in " + alert.DataSource)
 			if err != nil {
-				catcher.Error("error while getting incidents by pattern", err, map[string]any{})
+				catcher.Error("error while getting incidents by pattern", err, nil)
 				continue
 			}
 
@@ -50,7 +50,7 @@ func (p *Processor) processAlertToElastic() {
 						incidentExists = true
 						err = elastic.AddAlertToIncident(incident.ID, alert)
 						if err != nil {
-							catcher.Error("error while adding alert to incident", err, map[string]any{})
+							catcher.Error("error while adding alert to incident", err, nil)
 							continue
 						}
 					}
@@ -60,7 +60,7 @@ func (p *Processor) processAlertToElastic() {
 			if !incidentExists {
 				err = elastic.CreateNewIncident(alert)
 				if err != nil {
-					catcher.Error("error while creating incident", err, map[string]any{})
+					catcher.Error("error while creating incident", err, nil)
 					continue
 				}
 			}

soc-ai/processor/listener.go

@@ -42,14 +42,14 @@ func (p *Processor) handleAlerts(c *gin.Context) {
 	}
 
 	if !configurations.GetGPTConfig().ModuleActive {
-		catcher.Error("Droping request to /process, GPT module is not active", nil, map[string]any{})
+		catcher.Error("Droping request to /process, GPT module is not active", nil, nil)
 		c.JSON(http.StatusOK, "GPT module is not active")
 		return
 	}
 
 	var ids []string
 	if err := c.BindJSON(&ids); err != nil {
-		catcher.Error("error binding JSON", err, map[string]any{})
+		catcher.Error("error binding JSON", err, nil)
 		c.JSON(http.StatusBadRequest, gin.H{"error": err.Error()})
 		return
 	}
@@ -69,22 +69,22 @@ func (p *Processor) handleAlerts(c *gin.Context) {
 		var gptResponses []schema.GPTAlertResponse
 		err = json.Unmarshal(result, &gptResponses)
 		if err != nil {
-			catcher.Error("error decoding response from elastic", err, map[string]any{})
+			catcher.Error("error decoding response from elastic", err, nil)
 			c.JSON(http.StatusInternalServerError, fmt.Sprintf("error decoding response: %v", err))
 			return
 		}
 
 		if len(gptResponses) == 0 {
 			err = elastic.IndexStatus(id, "Processing", "create")
 			if err != nil {
-				catcher.Error("error creating doc in index", err, map[string]any{})
+				catcher.Error("error creating doc in index", err, nil)
 				c.JSON(http.StatusInternalServerError, fmt.Sprintf("error creating doc in index: %v", err))
 				return
 			}
 		} else {
 			err = elastic.IndexStatus(id, "Processing", "update")
 			if err != nil {
-				catcher.Error("error updating doc in index", err, map[string]any{})
+				catcher.Error("error updating doc in index", err, nil)
 				c.JSON(http.StatusInternalServerError, fmt.Sprintf("error updating doc in index: %v", err))
 				return
 			}

soc-ai/processor/processor.go

@@ -55,7 +55,7 @@ func (p *Processor) ProcessData() {
 func (p *Processor) RegisterError(message, id string) {
 	err := elastic.IndexStatus(id, "Error", "update")
 	if err != nil {
-		catcher.Error("error while indexing error in elastic", err, map[string]any{})
+		catcher.Error("error while indexing error in elastic", err, nil)
 	}
-	catcher.Error(message, nil, map[string]any{})
+	catcher.Error(message, nil, nil)
 }