Rewrite logic

zapier · Dec 9, 2024 · c85ef17 · c85ef17
1 parent be75f85
commit c85ef17
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/localdev/kubechecks/values.yaml b/localdev/kubechecks/values.yaml
@@ -1,6 +1,8 @@
 configMap:
   create: true
   env:
+    GRPC_ENFORCE_ALPN_ENABLED: false
+    KUBECHECKS_ALLOWED_NAMESPACES: httpbin,httpdump-a
     KUBECHECKS_LOG_LEVEL: debug
     KUBECHECKS_ENABLE_WEBHOOK_CONTROLLER: "false"
     KUBECHECKS_ARGOCD_API_INSECURE: "true"

diff --git a/pkg/config/config.go b/pkg/config/config.go
@@ -105,6 +105,11 @@ func NewWithViper(v *viper.Viper) (ServerConfig, error) {
 				return time.ParseDuration(input)
 			}
 
+			if in.String() == "string" && out.String() == "[]string" {
+				input := value.(string)
+				return []string{input}, nil
+			}
+
 			return value, nil
 		}
 	}); err != nil {

diff --git a/pkg/events/check.go b/pkg/events/check.go
@@ -292,16 +292,11 @@ func (ce *CheckEvent) Process(ctx context.Context) error {
 	ce.logger.Info().Msgf("adding %d apps to the queue", len(ce.affectedItems.Applications))
 	// Produce apps onto channel
 	for _, app := range ce.affectedItems.Applications {
-		if len(ce.ctr.Config.AllowedNamespaces) > 0 {
-			ns := strings.Split(ce.ctr.Config.AllowedNamespaces[0], ",")
-			if slices.Contains(ns, app.Spec.Destination.Namespace) {
-				ce.queueApp(app)
-			} else {
-				ce.logger.Info().Msgf("skipping app %s, namespace %s not allowed", app.Name, app.Spec.Destination.Namespace)
-			}
-		} else {
-			ce.queueApp(app)
+		if len(ce.ctr.Config.AllowedNamespaces) > 0 && !slices.Contains(ce.ctr.Config.AllowedNamespaces, app.Spec.Destination.Namespace) {
+			ce.logger.Info().Msgf("skipping app %s, namespace %s not allowed", app.Name, app.Spec.Destination.Namespace)
+			continue
 		}
+		ce.queueApp(app)
 	}
 
 	ce.wg.Wait()