fix: uri and request parsing;

src/extensions/nginx-app-protect/monitoring/processor/nap.go

@@ -104,6 +104,7 @@ var (
 		clientApplicationVersion,
 		transportProtocol,
 		httpURI,
+		request,
 	}
 )
 
@@ -450,11 +451,6 @@ func parseNAP(logEntry string, logger *logrus.Entry) (*NAPConfig, error) {
 		}
 	}
 
-	err := setValue(&waf, request, strings.Join(values[len(logFormatKeys):], ","), logger)
-	if err != nil {
-		return &NAPConfig{}, err
-	}
-
 	return &waf, nil
 }
 
@@ -506,13 +502,13 @@ func setValue(napConfig *NAPConfig, key, value string, logger *logrus.Entry) err
 	case httpServerPort:
 		napConfig.HTTPServerPort = value
 	case httpURI:
-		napConfig.HTTPURI = value
+		napConfig.HTTPURI = strings.ReplaceAll(value, "%2C", ",")
 	case isTruncated:
 		napConfig.IsTruncated = value
 	case policyName:
 		napConfig.PolicyName = value
 	case request:
-		napConfig.Request = value
+		napConfig.Request = strings.ReplaceAll(value, "%2C", ",")
 	case requestOutcome:
 		napConfig.RequestOutcome = value
 	case requestOutcomeReason:

src/plugins/nap_monitoring.go

@@ -55,7 +55,7 @@ func NewNAPMonitoring(env core.Environment, cfg *config.Config) (*NAPMonitoring,
 		log.Warnf("NAP Monitoring report count must be between %v and %v. Defaulting to %v",
 			minReportCountDelimiter,
 			maxReportCountDelimiter,
-			config.Defaults.NAPMonitoring.ReportInterval)
+			config.Defaults.NAPMonitoring.ReportCount)
 		cfg.NAPMonitoring.ReportCount = config.Defaults.NAPMonitoring.ReportCount
 	}