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

Author: JocLRojas

sophos/main.go

@@ -16,11 +16,11 @@ import (
 )
 
 func main() {
-	catcher.Info("Starting sophos central module...", map[string]any{})
+	catcher.Info("Starting sophos central 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.CHECKCON); err != nil {
-			catcher.Error("External connection failure detected", err, map[string]any{})
+			catcher.Error("External connection failure detected", err, nil)
 		}
 
 		endTime := time.Now().UTC()
@@ -45,10 +45,10 @@ func main() {
 		moduleConfig, err := client.GetUTMConfig(enum.SOPHOS)
 		if err != nil {
 			if strings.Contains(err.Error(), "invalid character '<'") {
-				catcher.Error("error getting configuration of the SOPHOS module: backend is not available", nil, map[string]any{})
+				catcher.Error("error getting configuration of the SOPHOS module: backend is not available", nil, nil)
 			}
 			if strings.TrimSpace(err.Error()) != "" {
-				catcher.Error("error getting configuration of the SOPHOS module", err, map[string]any{})
+				catcher.Error("error getting configuration of the SOPHOS module", err, nil)
 			}
 			continue
 		}

sophos/processor/processor.go

@@ -48,17 +48,17 @@ func (p *SophosCentralProcessor) getAccessToken() (string, error) {
 
 	response, _, err := utils.DoReq[map[string]any](configuration.AUTHURL, []byte(data.Encode()), http.MethodPost, headers)
 	if err != nil {
-		return "", catcher.Error("error making auth request", err, map[string]any{})
+		return "", catcher.Error("error making auth request", err, nil)
 	}
 
 	accessToken, ok := response["access_token"].(string)
 	if !ok || accessToken == "" {
-		return "", catcher.Error("access_token not found in response", nil, map[string]any{})
+		return "", catcher.Error("access_token not found in response", nil, nil)
 	}
 
 	expiresIn, ok := response["expires_in"].(float64)
 	if !ok {
-		return "", catcher.Error("expires_in not found in response", nil, map[string]any{})
+		return "", catcher.Error("expires_in not found in response", nil, nil)
 	}
 
 	p.AccessToken = accessToken
@@ -84,16 +84,16 @@ func (p *SophosCentralProcessor) getTenantInfo(accessToken string) error {
 
 	response, _, err := utils.DoReq[WhoamiResponse](configuration.WHOAMIURL, nil, http.MethodGet, headers)
 	if err != nil {
-		return catcher.Error("error making whoami request", err, map[string]any{})
+		return catcher.Error("error making whoami request", err, nil)
 	}
 
 	if response.ID == "" {
-		return catcher.Error("tenant ID not found in whoami response", nil, map[string]any{})
+		return catcher.Error("tenant ID not found in whoami response", nil, nil)
 	}
 	p.TenantID = response.ID
 
 	if response.ApiHosts.DataRegion == "" {
-		return catcher.Error("dataRegion not found in whoami response", nil, map[string]any{})
+		return catcher.Error("dataRegion not found in whoami response", nil, nil)
 	}
 	p.DataRegion = response.ApiHosts.DataRegion
 
@@ -122,12 +122,12 @@ type Pages struct {
 func (p *SophosCentralProcessor) getLogs(fromTime int64, nextKey string, group types.ModuleGroup) ([]TransformedLog, string, error) {
 	accessToken, err := p.getValidAccessToken()
 	if err != nil {
-		return nil, "", catcher.Error("error getting access token", err, map[string]any{})
+		return nil, "", catcher.Error("error getting access token", err, nil)
 	}
 
 	if p.TenantID == "" || p.DataRegion == "" {
 		if err := p.getTenantInfo(accessToken); err != nil {
-			return nil, "", catcher.Error("error getting tenant information", err, map[string]any{})
+			return nil, "", catcher.Error("error getting tenant information", err, nil)
 		}
 	}
 
@@ -138,7 +138,7 @@ func (p *SophosCentralProcessor) getLogs(fromTime int64, nextKey string, group t
 	for {
 		u, err := p.buildURL(fromTime, currentNextKey)
 		if err != nil {
-			return nil, "", catcher.Error("error building URL", err, map[string]any{})
+			return nil, "", catcher.Error("error building URL", err, nil)
 		}
 
 		headers := map[string]string{

sophos/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", err, map[string]any{})
+		return catcher.Error("error sending logs", err, nil)
 	}
 	defer resp.Body.Close()