ui/public/pages/helm/release.json (1)

352-371: 历史视图中的字段引用不一致。

历史视图仍在使用旧的嵌套字段引用如 chart.metadata.version 和 info.status，这些可能与新的简化 Release 模型不兼容，可能导致显示问题。

需要将这些字段更新为与新模型一致：

                      {
-                        "name": "chart.metadata.version",
+                        "name": "chart",
                        "label": "应用版本",
                        "type": "text"
                      },
                      {
-                        "name": "info.status",
+                        "name": "status",
                        "label": "状态",

pkg/models/helm_chart.go (1)

33-33: 修正注释中的错误实体类型。

注释中提到的 "InspectionCheckEvent" 应该是 "HelmChart"。

-// BatchSave 批量保存 InspectionCheckEvent 实例
+// BatchSave 批量保存 HelmChart 实例

pkg/controller/helm/helm.go (1)

19-28: 清理注释代码并确认架构变更。

从基于集群 REST 配置的方法切换到直接的 CLI 命令包装器是一个重大的架构变更。建议删除注释的代码而不是保留，以保持代码库的整洁。

-	// if namespace == "" {
-	// 	namespace = "default"
-	// }
-	// selectedCluster, err := amis.GetSelectedCluster(c)
-	// if err != nil {
-	// 	amis.WriteJsonError(c, err)
-	// 	return nil, err
-	// }
-	// restConfig := service.ClusterService().GetClusterByID(selectedCluster).GetRestConfig()
-	// h, err := helm.New(restConfig, namespace)
-	// return h, err
	cmd := helm.NewHelmCmd("helm")
	return cmd, nil

新的 CLI 方法简化了 Helm 客户端的创建过程。

pkg/helm/helm_cmd.go (1)

43-43: 使用结构化日志而非 fmt.Printf

建议使用 klog 进行日志记录，以保持与项目其他部分的一致性。

-	fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)
+	klog.V(6).Infof("[helm-cmd] exec: %s", cmdStr)

-	fmt.Printf("[helm-cmd] stdout: %s\n", stdout.String())
+	klog.V(6).Infof("[helm-cmd] stdout: %s", stdout.String())

-	fmt.Printf("[helm-cmd] stderr: %s\n", stderr.String())
+	klog.V(6).Infof("[helm-cmd] stderr: %s", stderr.String())

Also applies to: 53-58

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm_cmd.go (8)

39-60: 改进命令执行的安全性

当前实现使用 sh -c 执行命令，这可能导致命令注入漏洞。建议直接使用 exec.Command 而不是通过 shell。

此外，错误处理逻辑有问题：即使命令成功，如果有 stderr 输出也会返回错误。

 func (h *HelmCmd) runAndLog(args []string, stdin string) ([]byte, error) {
 	cmdStr := h.HelmBin + " " + strings.Join(args, " ")
 	fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)
-	// cmd := exec.Command(h.HelmBin, args...)
-	cmd := exec.Command("sh", "-c", cmdStr)
+	cmd := exec.Command(h.HelmBin, args...)
 	if stdin != "" {
 		cmd.Stdin = strings.NewReader(stdin)
 	}
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr
 	err := cmd.Run()
 	if stdout.Len() > 0 {
 		fmt.Printf("[helm-cmd] stdout: %s\n", stdout.String())
 	}
 	if stderr.Len() > 0 {
 		fmt.Printf("[helm-cmd] stderr: %s\n", stderr.String())
-		err = fmt.Errorf(stderr.String())
+		// 只有在命令执行失败时才将 stderr 作为错误
+		if err != nil {
+			err = fmt.Errorf("command failed: %v, stderr: %s", err, stderr.String())
+		}
 	}
 	return stdout.Bytes(), err
 }

---

42-42: 避免记录敏感信息

当前代码会将包含密码的完整命令记录到日志中，这是一个安全风险。

 func (h *HelmCmd) runAndLog(args []string, stdin string) ([]byte, error) {
-	cmdStr := h.HelmBin + " " + strings.Join(args, " ")
-	fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)
+	// 构建安全的日志字符串，隐藏敏感参数
+	safeArgs := make([]string, len(args))
+	copy(safeArgs, args)
+	for i := 0; i < len(safeArgs)-1; i++ {
+		if safeArgs[i] == "--password" {
+			safeArgs[i+1] = "***"
+		}
+	}
+	cmdStr := h.HelmBin + " " + strings.Join(safeArgs, " ")
+	fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)

---

87-87: 避免直接使用 os.Setenv

在并发环境中，os.Setenv 会影响整个进程的环境变量，可能导致竞态条件。建议为每个命令设置独立的环境变量。

建议修改 runAndLog 方法以支持环境变量：

func (h *HelmCmd) runAndLogWithEnv(args []string, stdin string, env map[string]string) ([]byte, error) {
    cmd := exec.Command(h.HelmBin, args...)
    cmd.Env = os.Environ()
    for k, v := range env {
        cmd.Env = append(cmd.Env, fmt.Sprintf("%s=%s", k, v))
    }
    // ... 其余代码
}

---

106-111: 返回值顺序不符合 Go 惯例

Go 的惯例是将 error 作为最后一个返回值。当前实现返回 (error, bool)，应该改为 (bool, error)。

-func (h *HelmCmd) updateRepoByName(repoEntry *repo.Entry, helmRepo *models.HelmRepository) (error, bool) {
+func (h *HelmCmd) updateRepoByName(repoEntry *repo.Entry, helmRepo *models.HelmRepository) (bool, error) {
 	// 4. helm repo update
 	out, err := h.runAndLog([]string{"repo", "update", repoEntry.Name}, "")
 	if err != nil {
-		return fmt.Errorf("helm repo update failed: %v, output: %s", err, string(out)), true
+		return true, fmt.Errorf("helm repo update failed: %v, output: %s", err, string(out))
 	}

相应地更新调用处：

-	err2, done := h.updateRepoByName(repoEntry, helmRepo)
+	done, err2 := h.updateRepoByName(repoEntry, helmRepo)

---

127-127: 使用 repository_id 而非 repository_name 删除记录

当前使用 repository_name 删除记录可能不够精确，特别是当存在同名但不同 URL 的仓库时。建议使用 repository_id 以确保准确性。

-	dao.DB().Where("repository_name = ?", repoEntry.Name).Delete(models.HelmChart{})
+	dao.DB().Where("repository_id = ?", helmRepo.ID).Delete(&models.HelmChart{})

---

217-217: UpgradeRelease 方法中的 chartRef 构造错误

使用 releaseName 作为 chart 名称是错误的。应该先获取 release 的历史记录，从中提取正确的 chart 名称。

 func (h *HelmCmd) UpgradeRelease(releaseName, repoName, targetVersion string, values ...string) error {
-	chartRef := fmt.Sprintf("%s/%s", repoName, releaseName)
+	// 需要先获取 release 信息以确定 chart 名称
+	// 注意：当前方法签名缺少 namespace 参数
+	// 建议修改方法签名以包含 namespace
 	args := []string{"upgrade", releaseName, chartRef, "--version", targetVersion}

注意：此方法签名与接口定义不一致，接口中没有 namespace 参数，但实际执行 helm 命令时需要指定 namespace。

---

243-243: 移除硬编码的 stderr 重定向

在命令参数中硬编码 2>/dev/null 依赖于 shell 特性，这与之前建议的直接使用 exec.Command 不兼容。

-	args := []string{"search", "repo", chartRef, "-o", "json", "--versions", "2>/dev/null"}
+	args := []string{"search", "repo", chartRef, "-o", "json", "--versions"}

---

289-289: 处理 updateRepoByName 的返回值

调用 updateRepoByName 但忽略了其返回值，应该处理可能的错误。

-		h.updateRepoByName(repoEntry, item)
+		done, err := h.updateRepoByName(repoEntry, item)
+		if err != nil {
+			klog.Errorf("Failed to update repo %s: %v", item.Name, err)
+		}

pkg/helm/helm.go (1)

8-539: 建议删除注释代码而非保留

整个 SDK 实现已被注释，既然已经切换到二进制命令方式，建议完全删除这些注释代码，以保持代码库的整洁。如果需要保留历史记录，Git 版本控制已经提供了这个功能。

-// import (
-//
-//	"errors"
-//	"fmt"
-...
-// func debug(format string, v ...interface{}) {
-// 	klog.V(6).Infof(format, v...)
-// }

pkg/helm/helm_cmd.go (1)

128-172: 优化批量保存逻辑

当前实现逐个保存 chart，对于包含大量 chart 的仓库可能效率较低。建议使用批量保存来提高性能。

+	var charts []models.HelmChart
 	for chartName, versionList := range index.Entries {
 		if len(versionList) == 0 {
 			continue
 		}
 		slice.SortBy(versionList, func(a *repo.ChartVersion, b *repo.ChartVersion) bool {
 			return a.Created.After(b.Created)
 		})
 		ct := versionList[0]
 		if ct.Metadata != nil {
 			m := models.HelmChart{
 				RepositoryID:   helmRepo.ID,
 				RepositoryName: helmRepo.Name,
 				Name:           chartName,
 				// ... 其他字段
 			}
-			err = m.Save(nil)
-			if err != nil {
-				fmt.Printf("[helm-cmd] warn: save helm chart to database error: %v\n", err)
-			}
+			charts = append(charts, m)
 		}
 	}
+	// 批量保存
+	if len(charts) > 0 {
+		if err := dao.DB().CreateInBatches(charts, 100).Error; err != nil {
+			return fmt.Errorf("batch save helm charts error: %v", err)
+		}
+	}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm_cmd.go (7)

42-42: 避免在日志中记录敏感信息

当前代码会将完整的命令字符串打印到日志中，这可能包含密码等敏感信息。

---

45-45: 严重安全问题：命令注入风险

使用 sh -c 执行拼接的命令字符串存在命令注入漏洞。攻击者可能通过特殊构造的输入执行任意命令。

---

114-115: 潜在的路径遍历安全风险

直接使用用户输入的 repoEntry.Name 构造文件路径可能导致路径遍历攻击。

---

217-217: 错误：使用 releaseName 构造 chartRef

使用 releaseName 作为 chart 名称是错误的。Release 名称和 Chart 名称可能不同。

---

243-243: 移除硬编码的 stderr 重定向

在命令参数中硬编码 2>/dev/null 不会生效，因为这是 shell 特性。

---

289-289: 处理 updateRepoByName 的错误返回值

当前代码忽略了 updateRepoByName 的错误返回值，可能导致更新失败时没有任何提示。

建议修改：

-		_, _ = h.updateRepoByName(repoEntry, item)
+		_, err := h.updateRepoByName(repoEntry, item)
+		if err != nil {
+			klog.Errorf("Failed to update repo %s: %v", item.Name, err)
+		}

---

40-63: 改进命令执行的安全性和错误处理

当前实现存在多个问题需要修复：

1. 使用 sh -c 执行命令存在命令注入风险
2. 错误处理逻辑不正确：stderr 有输出不一定表示错误

建议修改如下：

 func (h *HelmCmd) runAndLog(args []string, stdin string) ([]byte, error) {
 	cmdStr := h.HelmBin + " " + strings.Join(args, " ")
-	fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)
-	// cmd := exec.Command(h.HelmBin, args...)
-
-	cmd := exec.Command("sh", "-c", cmdStr)
+	// 移除敏感信息的日志记录
+	klog.V(6).Infof("[helm-cmd] exec: %s %s", h.HelmBin, maskSensitiveArgs(args))
+	cmd := exec.Command(h.HelmBin, args...)
 	cmd.Env = append(os.Environ(), fmt.Sprintf("%s=%s", "HELM_CACHE_HOME", h.repoCacheDir))
 
 	if stdin != "" {
 		cmd.Stdin = strings.NewReader(stdin)
 	}
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr
 	err := cmd.Run()
 	if stdout.Len() > 0 {
-		fmt.Printf("[helm-cmd] stdout: %s\n", stdout.String())
+		klog.V(6).Infof("[helm-cmd] stdout: %s", stdout.String())
 	}
 	if stderr.Len() > 0 {
-		fmt.Printf("[helm-cmd] stderr: %s\n", stderr.String())
-		err = fmt.Errorf(stderr.String())
+		klog.V(6).Infof("[helm-cmd] stderr: %s", stderr.String())
+		// 只有在命令执行失败时才将 stderr 作为错误
+		if err != nil {
+			err = fmt.Errorf("command failed: %v, stderr: %s", err, stderr.String())
+		}
 	}
 	return stdout.Bytes(), err
 }
+
+// maskSensitiveArgs 隐藏命令参数中的敏感信息
+func maskSensitiveArgs(args []string) []string {
+	masked := make([]string, len(args))
+	copy(masked, args)
+	for i := 0; i < len(masked); i++ {
+		if i > 0 && (masked[i-1] == "--password" || masked[i-1] == "--username") {
+			masked[i] = "***"
+		}
+	}
+	return masked
+}

pkg/helm/helm_cmd.go (2)

88-94: 建议使用更安全的方式传递凭据

当前通过命令行参数传递用户名和密码，这些信息可能被系统进程列表（如 ps 命令）暴露。

考虑使用环境变量或配置文件传递敏感信息：

 args := []string{"repo", "add", repoEntry.Name, repoEntry.URL}
-if repoEntry.Username != "" {
-	args = append(args, "--username", repoEntry.Username)
-}
-if repoEntry.Password != "" {
-	args = append(args, "--password", repoEntry.Password)
-}
+// 使用配置文件传递凭据
+if repoEntry.Username != "" || repoEntry.Password != "" {
+	configFile, err := h.createTempRepoConfig(repoEntry)
+	if err != nil {
+		return fmt.Errorf("create temp config failed: %v", err)
+	}
+	defer os.Remove(configFile)
+	args = append(args, "--repo-config", configFile)
+}

---

295-326: 代码重复：考虑提取通用的 JSON 命令执行方法

多个方法（GetReleaseList、GetReleaseNote、GetReleaseValues 等）都遵循相同的模式：执行命令、获取 JSON 输出、解析结果。

建议提取一个通用方法减少代码重复：

// runJSONCommand 执行 helm 命令并返回 JSON 输出
func (h *HelmCmd) runJSONCommand(args []string) ([]byte, error) {
    args = append(args, "-o", "json")
    out, err := h.runAndLog(args, "")
    if err != nil {
        return nil, fmt.Errorf("helm %s failed: %v, output: %s", args[0], err, string(out))
    }
    return out, nil
}

// 使用示例
func (h *HelmCmd) GetReleaseList() ([]*models.Release, error) {
    out, err := h.runJSONCommand([]string{"list", "-A"})
    if err != nil {
        return nil, err
    }
    var releases []*models.Release
    if err := json.Unmarshal(out, &releases); err != nil {
        return nil, fmt.Errorf("unmarshal helm list output failed: %v", err)
    }
    return releases, nil
}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm.go (1)

8-539: 建议删除注释代码而非保留

整个 SDK 实现已被注释掉。根据 PR 描述，这是从 SDK 迁移到二进制命令方式的一部分。建议完全删除这些注释代码，以保持代码库整洁。

如果确认不再需要 SDK 实现，建议：

1. 删除所有注释代码（第8-539行）
2. 在文件顶部添加说明，指出现在使用 HelmCmd 实现
3. 或者将这些代码移到单独的废弃文件中以备参考

pkg/helm/helm_cmd.go (5)

288-288: 处理 updateRepoByName 的错误

虽然使用 _, _ 显式忽略了返回值，但建议至少记录错误信息以便调试。

---

40-63: 严重安全问题：命令注入风险

使用 sh -c 执行命令存在命令注入漏洞。攻击者可能通过特殊构造的参数执行任意命令。

必须修改为直接使用 exec.Command：

 func (h *HelmCmd) runAndLog(args []string, stdin string) ([]byte, error) {
     cmdStr := h.HelmBin + " " + strings.Join(args, " ")
-    fmt.Printf("[helm-cmd] exec: %s\n", cmdStr)
-    // cmd := exec.Command(h.HelmBin, args...)
-
-    cmd := exec.Command("sh", "-c", cmdStr)
+    // 只记录命令用于调试，但要注意不记录敏感信息
+    safeArgs := make([]string, len(args))
+    copy(safeArgs, args)
+    for i, arg := range safeArgs {
+        if i > 0 && (args[i-1] == "--password" || args[i-1] == "--username") {
+            safeArgs[i] = "***"
+        }
+    }
+    fmt.Printf("[helm-cmd] exec: %s %s\n", h.HelmBin, strings.Join(safeArgs, " "))
+    
+    cmd := exec.Command(h.HelmBin, args...)
     cmd.Env = append(os.Environ(), fmt.Sprintf("%s=%s", "HELM_CACHE_HOME", h.repoCacheDir))
 
     if stdin != "" {
         cmd.Stdin = strings.NewReader(stdin)
     }
     var stdout, stderr bytes.Buffer
     cmd.Stdout = &stdout
     cmd.Stderr = &stderr
     err := cmd.Run()
     if stdout.Len() > 0 {
         fmt.Printf("[helm-cmd] stdout: %s\n", stdout.String())
     }
     if stderr.Len() > 0 {
         fmt.Printf("[helm-cmd] stderr: %s\n", stderr.String())
-        err = fmt.Errorf(stderr.String())
+        // 只在命令失败时将 stderr 作为错误信息
+        if err != nil {
+            err = fmt.Errorf("command failed: %v, stderr: %s", err, stderr.String())
+        }
     }
     return stdout.Bytes(), err
 }

---

88-94: 避免在日志中暴露敏感信息

当前实现会在日志中记录密码等敏感信息。

参见第40-63行的修复建议，其中包含了对敏感参数的处理。

---

114-115: 验证仓库名称以防止路径遍历

虽然路径是在缓存目录内构建的，但仍应验证仓库名称以防止路径遍历攻击。

添加仓库名称验证：

+    // 验证仓库名称，防止路径遍历
+    if strings.Contains(repoEntry.Name, "..") || strings.Contains(repoEntry.Name, "/") {
+        return false, fmt.Errorf("invalid repository name: %s", repoEntry.Name)
+    }
+    
     cachePath := fmt.Sprintf("%s/repository/%s-index.yaml", h.repoCacheDir, repoEntry.Name)

---

242-242: 移除硬编码的 stderr 重定向

在命令参数中包含 2>/dev/null 依赖于 shell 特性，与建议的直接使用 exec.Command 不兼容。

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm_cmd.go (4)

114-115: 验证文件路径以防止路径遍历攻击。

直接使用用户提供的仓库名称构建文件路径可能存在路径遍历风险。

建议添加路径验证：

+import "path/filepath"
+
+// 验证仓库名称
+if strings.Contains(repoEntry.Name, "..") || strings.Contains(repoEntry.Name, "/") {
+    return false, fmt.Errorf("invalid repository name: %s", repoEntry.Name)
+}
 cachePath := fmt.Sprintf("%s/repository/%s-index.yaml", h.repoCacheDir, repoEntry.Name)
+cachePath = filepath.Clean(cachePath)

---

42-42: 避免记录敏感信息。

命令字符串可能包含密码等敏感信息，不应该直接打印到日志中。

---

45-45: 避免使用 shell 执行命令，存在命令注入风险。

使用 sh -c 执行命令可能导致命令注入漏洞。建议直接使用 exec.Command(h.HelmBin, args...)。

建议修改：

-cmd := exec.Command("sh", "-c", cmdStr)
+cmd := exec.Command(h.HelmBin, args...)

---

269-269: 移除硬编码的 stderr 重定向。

在命令参数中硬编码 2>/dev/null 依赖于 shell 特性，与直接使用 exec.Command 不兼容。

-args := []string{"search", "repo", chartRef, "-o", "json", "--versions", "2>/dev/null"}
+args := []string{"search", "repo", chartRef, "-o", "json", "--versions"}

pkg/helm/helm_cmd.go (1)

315-315: 处理 updateRepoByName 的错误返回值。

虽然使用了 _, _ 明确忽略返回值，但建议至少记录错误以便调试。

-_, _ = h.updateRepoByName(repoEntry, item)
+_, err := h.updateRepoByName(repoEntry, item)
+if err != nil {
+    klog.Errorf("Failed to update repo %s: %v", item.Name, err)
+}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

ui/src/components/Amis/custom/Helm/HelmViewRelease.tsx (4)

18-18: 未使用的 ref 参数

组件使用了 React.forwardRef 但没有实际使用 ref，建议移除 forwardRef 包装或明确使用 ref。

-const HelmViewRelease = React.forwardRef<HTMLSpanElement, HelmViewReleaseProps>(({ data }, _) => {
+const HelmViewRelease: React.FC<HelmViewReleaseProps> = ({ data }) => {

---

21-24: 变量命名不一致

变量命名存在不一致问题，建议统一命名风格以提高代码可读性。

-    let chartName = data.chart
-    let releaseName = data.name
-    let namespace = data.namespace
-    let revision = data.revision
+    const chartName = data.chart;
+    const releaseName = data.name;
+    const namespace = data.namespace;
+    const revision = data.revision;

---

61-61: 建议将内联样式提取为 CSS 类

内联样式不利于维护和重用，建议提取为 CSS 类或使用 styled-components。

可以在组件文件同目录创建对应的 CSS 文件：

.editor-container {
    border: 1px solid #d9d9d9;
    border-radius: 4px;
}

然后在组件中使用：

-                    <div style={{ border: '1px solid #d9d9d9', borderRadius: '4px' }}>
+                    <div className="editor-container">

---

63-63: 编辑器高度计算可能不够灵活

硬编码的高度计算 calc(100vh - 200px) 可能在不同布局下不够灵活，建议考虑更动态的高度计算方式。

可以考虑使用 useRef 和 ResizeObserver 来动态调整编辑器高度，或者将高度作为 props 传入以增加组件的灵活性。

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm_cmd.go (9)

48-48: kubeconfig 文件权限过于宽松

kubeconfig 包含敏感认证信息，但使用 0644 权限允许其他用户读取。

修改文件权限为仅所有者可读写：

-if err := os.WriteFile(kubeconfigPath, []byte(config), 0644); err != nil {
+if err := os.WriteFile(kubeconfigPath, []byte(config), 0600); err != nil {

---

68-68: 命令注入安全漏洞

使用 sh -c 执行命令存在命令注入风险。应直接使用 exec.Command 执行。

修改为直接执行：

-cmd := exec.Command("sh", "-c", cmdStr)
+cmd := exec.Command(h.HelmBin, args...)

---

84-87: 错误处理逻辑有误

即使命令成功执行，如果有 stderr 输出也会返回错误。某些命令正常运行时也会输出到 stderr。

修正错误处理：

 if stderr.Len() > 0 {
     klog.V(6).Infof("[helm-cmd] stderr: %s\n", stderr.String())
-    err = fmt.Errorf("%s", stderr.String())
+    // 只有在命令执行失败时才将 stderr 作为错误
+    if err != nil {
+        err = fmt.Errorf("command failed: %v, stderr: %s", err, stderr.String())
+    }
 }

---

116-120: 避免记录敏感信息

密码通过命令行参数传递并在日志中以明文形式记录，存在安全风险。

建议使用环境变量或标准输入传递密码：

 if repoEntry.Username != "" {
     args = append(args, "--username", repoEntry.Username)
 }
 if repoEntry.Password != "" {
-    args = append(args, "--password", repoEntry.Password)
+    // 通过标准输入传递密码，避免在命令行中暴露
+    args = append(args, "--password-stdin")
 }
-out, err := h.runAndLog(args, "")
+out, err := h.runAndLog(args, repoEntry.Password)

---

132-132: 返回值顺序不符合 Go 惯例

Go 惯例是将 error 作为最后一个返回值。当前返回 (bool, error)。

修改返回值顺序并更新所有调用处：

-func (h *HelmCmd) updateRepoByName(repoEntry *repo.Entry, helmRepo *models.HelmRepository) (bool, error) {
+func (h *HelmCmd) updateRepoByName(repoEntry *repo.Entry, helmRepo *models.HelmRepository) (bool, error) {

---

250-250: Values 可能包含敏感信息

将完整的 values 内容存储到数据库可能暴露密码、密钥等敏感信息。

建议实施以下安全措施之一：

• 对敏感字段进行脱敏处理
• 加密存储 values 内容
• 仅存储 values 的哈希值用于变更检测
• 使用专门的凭据管理服务

---

296-296: 移除硬编码的 stderr 重定向

在命令参数中硬编码 2>/dev/null 依赖于 shell 特性，与直接使用 exec.Command 不兼容。

-args := []string{"search", "repo", chartRef, "-o", "json", "--versions", "2>/dev/null"}
+args := []string{"search", "repo", chartRef, "-o", "json", "--versions"}

---

342-342: 处理 updateRepoByName 的返回值

调用 updateRepoByName 但忽略了其返回值，应该处理可能的错误。

-_, _ = h.updateRepoByName(repoEntry, item)
+if _, err := h.updateRepoByName(repoEntry, item); err != nil {
+    klog.Errorf("Failed to update repo %s: %v", item.Name, err)
+}

---

43-43: 存在路径遍历安全漏洞

cluster 参数未经验证直接用于构建文件路径。如果包含 ../ 等路径遍历序列，可能导致文件被写入任意位置。

建议添加输入验证：

+import "path/filepath"

 kubeconfigPath := fmt.Sprintf("%s/%s-kubeconfig.yaml", repoCacheDir, strings.ReplaceAll(cluster, "/", "|"))
+// 验证路径安全性
+kubeconfigPath = filepath.Clean(kubeconfigPath)
+if !strings.HasPrefix(kubeconfigPath, repoCacheDir) {
+    return nil, fmt.Errorf("invalid cluster name")
+}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

Dockerfile (1)

2-7: ARG 参数未一一声明，可能导致构建警告

-ldflags 中引用了 $GIT_TAG, $GIT_REPOSITORY, $BUILD_DATE 但前面只声明了少部分 ARG。未声明的变量在构建时为空并触发 unused build-arg 警告，亦降低可读性。请补充声明或移除未使用的变量。

Dockerfile.action (2)

30-31: 避免在运行镜像里执行 apk upgrade

apk upgrade 会把基础镜像的全部包都升级到 edge 仓库最新版本，可能导致：

• 镜像尺寸暴涨；
• 与基础镜像安全扫描基线不一致；
• 难以重现的“雪花”环境。

除非确有必要安装特定补丁，建议仅使用 apk add --no-cache <pkg> 精确安装所需包。

---

32-33: 使用 COPY 替代 ADD 以符合最佳实践

此处只是把本地脚本放进镜像，ADD 的额外功能（自动解压 / 远程 URL）并未用到，容易误导阅读者，也触发 Hadolint DL3020。

-ADD reload.sh /app/reload.sh
+COPY reload.sh /app/reload.sh

Dockerfile (2)

16-19: apk upgrade 同样应避免

和 Dockerfile.action 中的情况一致，完整升级会引入不可控风险。仅安装所需的软件包即可，建议拆分为：

-RUN sed -i 's/dl-cdn.alpinelinux.org/mirrors.aliyun.com/g' /etc/apk/repositories \
-    && apk upgrade && apk add --no-cache curl bash inotify-tools alpine-conf busybox-extras tzdata   tar gzip\
+RUN sed -i 's/dl-cdn.alpinelinux.org/mirrors.aliyun.com/g' /etc/apk/repositories \
+    && apk add --no-cache curl bash inotify-tools busybox-extras tzdata tar gzip \

---

20-20: 将 ADD reload.sh 改为 COPY

功能一致但 COPY 更语义化，亦可消除 Hadolint 警告。

-ADD reload.sh /app/reload.sh
+COPY reload.sh /app/reload.sh

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

pkg/helm/helm_cmd.go (8)

73-130: 改进命令执行的安全性和错误处理

---

103-103: 避免记录敏感信息

---

173-173: 返回值顺序不符合 Go 惯例

---

182-182: 路径遍历安全风险

---

194-194: 使用 repository_id 而非 repository_name 删除记录

---

273-299: 存储敏感数据的安全考虑

---

337-337: 移除硬编码的 stderr 重定向

---

383-383: 处理 updateRepoByName 的返回值

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro