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

Author: JocLRojas

office365/main.go

@@ -16,11 +16,11 @@ import (
 )
 
 func main() {
-	catcher.Info("Starting O365 module", map[string]any{})
+	catcher.Info("Starting O365 module", nil)
 	intKey := configuration.GetInternalKey()
 	panelServ := configuration.GetPanelServiceName()
 	if intKey == "" || panelServ == "" {
-		catcher.Error("Internal key or panel service name is not set. Exiting...", nil, map[string]any{})
+		catcher.Error("Internal key or panel service name is not set. Exiting...", nil, nil)
 		os.Exit(1)
 	}
 	client := utmconf.NewUTMClient(intKey, "http://"+panelServ)
@@ -33,7 +33,7 @@ func main() {
 
 	for range ticker.C {
 		if err := utils.ConnectionChecker(configuration.LoginUrl); err != nil {
-			catcher.Error("External connection failure detected", err, map[string]any{})
+			catcher.Error("External connection failure detected", err, nil)
 		}
 
 		endTime := time.Now().UTC()
@@ -43,10 +43,10 @@ func main() {
 		moduleConfig, err := client.GetUTMConfig(enum.O365)
 		if err != nil {
 			if strings.Contains(err.Error(), "invalid character '<'") {
-				catcher.Error("error getting configuration of the O365 module: backend is not available", nil, map[string]any{})
+				catcher.Error("error getting configuration of the O365 module: backend is not available", nil, nil)
 			}
 			if strings.TrimSpace(err.Error()) != "" {
-				catcher.Error("error getting configuration of the O365 module", err, map[string]any{})
+				catcher.Error("error getting configuration of the O365 module", err, nil)
 			}
 			continue
 		}

office365/processor/processor.go

@@ -137,23 +137,23 @@ func (o *OfficeProcessor) GetLogs(startTime time.Time, endTime time.Time, group
 	for _, subscription := range o.Subscriptions {
 		contentList, err := o.GetContentList(subscription, startTime, endTime, group)
 		if err != nil {
-			catcher.Error("error getting content list", err, map[string]any{})
+			catcher.Error("error getting content list", err, nil)
 			continue
 		}
 		logsCounter := 0
 		if len(contentList) > 0 {
 			for _, log := range contentList {
 				details, err := o.GetContentDetails(log.ContentUri)
 				if err != nil {
-					catcher.Error("error getting content details", err, map[string]any{})
+					catcher.Error("error getting content details", err, nil)
 					continue
 				}
 				if len(details) > 0 {
 					logsCounter += len(details)
 					cleanLogs := ETLProcess(details, group)
 					err := SendToLogstash(cleanLogs)
 					if err != nil {
-						catcher.Error("error sending logs to logstash", err, map[string]any{})
+						catcher.Error("error sending logs to logstash", err, nil)
 						continue
 					}
 				}

office365/processor/pull.go

@@ -14,13 +14,13 @@ func PullLogs(startTime time.Time, endTime time.Time, group types.ModuleGroup) {
 
 	err := agent.GetAuth()
 	if err != nil {
-		catcher.Error("error getting auth token", err, map[string]any{})
+		catcher.Error("error getting auth token", err, nil)
 		return
 	}
 
 	err = agent.StartSubscriptions()
 	if err != nil {
-		catcher.Error("error starting subscriptions", err, map[string]any{})
+		catcher.Error("error starting subscriptions", err, nil)
 		return
 	}
 

office365/processor/sendData.go

@@ -29,11 +29,11 @@ func SendToLogstash(data []TransformedLog) *logger.Error {
 	for _, str := range data {
 		body, err := json.Marshal(str)
 		if err != nil {
-			catcher.Error("error encoding log to JSON", err, map[string]any{})
+			catcher.Error("error encoding log to JSON", err, nil)
 			continue
 		}
 		if err := sendLogs(body); err != nil {
-			catcher.Error("error sending logs to logstach", err, map[string]any{})
+			catcher.Error("error sending logs to logstach", err, nil)
 			continue
 		}
 	}
@@ -45,12 +45,12 @@ func sendLogs(log []byte) error {
 
 	req, err := http.NewRequest("POST", url, bytes.NewBuffer(log))
 	if err != nil {
-		return catcher.Error("error creating request", err, map[string]any{})
+		return catcher.Error("error creating request", err, nil)
 	}
 
 	resp, err := client.Do(req)
 	if err != nil {
-		return catcher.Error("error sending logs: %v", err, map[string]any{})
+		return catcher.Error("error sending logs: %v", err, nil)
 	}
 	defer resp.Body.Close()