PR feedback

Author: aphralG

internal/collector/otel_collector_plugin.go

@@ -724,8 +724,6 @@ func (oc *Collector) findAvailableSyslogServer(ctx context.Context, napSyslogSer
 		slog.DebugContext(ctx, "Failed to close syslog server", "address", napSyslogServer, "error", closeError)
 	}
 
-	slog.InfoContext(ctx, "Found valid NAP syslog server", "address", napSyslogServer)
-
 	oc.previousNAPSysLogServer = napSyslogServer
 
 	return napSyslogServer

internal/datasource/config/nginx_config_parser.go

@@ -250,12 +250,6 @@ func (ncp *NginxConfigParser) createNginxConfigContext(
 			nginxConfigContext.PlusAPIs = append(nginxConfigContext.PlusAPIs, plusAPIs...)
 		}
 
-		if napEnabled && nginxConfigContext.NAPSysLogServer == "" {
-			slog.WarnContext(ctx, fmt.Sprintf("Could not find available local NGINX App Protect syslog"+
-				" server configured on port %s. Security violations will not be collected.",
-				ncp.agentConfig.SyslogServer.Port))
-		}
-
 		fileMeta, err := files.FileMeta(conf.File)
 		if err != nil {
 			slog.WarnContext(ctx, "Unable to get file metadata", "file_name", conf.File, "error", err)
@@ -264,6 +258,15 @@ func (ncp *NginxConfigParser) createNginxConfigContext(
 		}
 	}
 
+	if napEnabled && nginxConfigContext.NAPSysLogServer == "" {
+		slog.WarnContext(ctx, fmt.Sprintf("Could not find available local NGINX App Protect syslog"+
+			" server configured on port %s. Security violations will not be collected.",
+			ncp.agentConfig.SyslogServer.Port))
+	} else {
+		slog.InfoContext(ctx, fmt.Sprintf("Found available local NGINX App Protect syslog"+
+			"server configured on port %s", ncp.agentConfig.SyslogServer.Port))
+	}
+
 	nginxConfigContext.StubStatus = ncp.FindStubStatusAPI(ctx, nginxConfigContext)
 	nginxConfigContext.PlusAPI = ncp.FindPlusAPI(ctx, nginxConfigContext)