pkg/controller/helm/helm_repo.go (3)

86-113: DeleteRepo 方法缺少权限检查

与 AddOrUpdateRepo 类似，这个方法允许任何用户删除 Helm 仓库，存在严重的安全风险。

建议在删除操作前添加权限检查：

 func (r *HelmRepoController) DeleteRepo(c *gin.Context) {
     ids := c.Param("ids")
+
+    // 检查权限
+    _, _, err := handleCommonLogic(c, "delete", "", "", "helm-repo")
+    if err != nil {
+        amis.WriteJsonError(c, err)
+        return
+    }

     h, err := getHelmWithNoCluster()

---

114-130: UpdateReposIndex 方法缺少权限检查

更新仓库索引操作同样需要权限控制。

建议添加权限检查：

 func (r *HelmRepoController) UpdateReposIndex(c *gin.Context) {
     var req struct {
         IDs string `json:"ids"`
     }
     if err := c.ShouldBindJSON(&req); err != nil {
         amis.WriteJsonError(c, err)
         return
     }
+
+    // 检查权限
+    _, _, err := handleCommonLogic(c, "update", "", "", "helm-repo")
+    if err != nil {
+        amis.WriteJsonError(c, err)
+        return
+    }

     h, err := getHelmWithNoCluster()

---

39-57: 在 AddOrUpdateRepo 方法中添加权限校验

当前 pkg/controller/helm/helm_repo.go 的 AddOrUpdateRepo 无权限校验，与 helm_release.go 中对其他 Helm 操作均调用 handleCommonLogic 不一致，请在调用 getHelmWithNoCluster() 之前加入如下校验：

diff --git a/pkg/controller/helm/helm_repo.go b/pkg/controller/helm/helm_repo.go
index  abcdef1..1234567 100644
--- a/pkg/controller/helm/helm_repo.go
+++ b/pkg/controller/helm/helm_repo.go
@@ -39,6 +39,12 @@ func (r *HelmRepoController) AddOrUpdateRepo(c *gin.Context) {
     if err := c.ShouldBindJSON(&repo); err != nil {
         amis.WriteJsonError(c, err)
         return
     }
+
+    // 权限校验：创建或更新 Helm 仓库
+    _, _, err := handleCommonLogic(c, "create", repo.Name, "", "helm-repo")
+    if err != nil {
+        amis.WriteJsonError(c, err)
+        return
+    }
 
     h, err := getHelmWithNoCluster()
     if err != nil {
         amis.WriteJsonError(c, err)
         return
     }

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

37-56: GetChartValue 方法缺少权限检查

获取 Chart 值的操作应该有权限控制，确保只有授权用户才能访问。

建议添加权限检查：

 func (hc *HelmChartController) GetChartValue(c *gin.Context) {
     repoName := c.Param("repo")
     chartName := c.Param("chart")
     version := c.Param("version")
+
+    // 检查权限
+    _, _, err := handleCommonLogic(c, "get", chartName, "", repoName)
+    if err != nil {
+        amis.WriteJsonError(c, err)
+        return
+    }
+
     h, err := getHelmWithNoCluster()

pkg/cb/cb.go (1)

70-73: 可能存在命名空间重复的问题

如果 ns 已经存在于 nsList 中，这里的追加操作会导致重复。建议在追加前检查是否已存在。

 ns := stmt.Namespace
 if ns != "" {
-    nsList = append(nsList, ns)
+    if !slice.Contain(nsList, ns) {
+        nsList = append(nsList, ns)
+    }
 }

pkg/comm/permission.go (2)

184-186: 日志格式可能有误

日志中 roleString=%v 但实际记录的是 roles，变量名不一致可能造成混淆。

-    klog.V(6).Infof("cb: cluster= %s,user= %s, role= %s，roleString=%v, operation=%s,  resource=[%s/%s] ",
-        cluster, username, roleString, roles, action, ns, name)
+    klog.V(6).Infof("cb: cluster=%s, user=%s, platformRole=%s, clusterRoles=%v, operation=%s, resource=[%s/%s]",
+        cluster, username, roleString, roles, action, ns, name)

---

18-187: 权限检查逻辑实现完善

新的集中式权限检查函数很好地实现了：

• 平台管理员的特殊处理
• 基于角色的访问控制
• 命名空间白名单和黑名单机制
• 详细的错误信息

建议考虑将来进一步模块化，将不同 action 的处理逻辑抽取为独立函数以提高可维护性。

pkg/controller/helm/helm_release.go (2)

222-254: 批量卸载操作可以优化

在循环内创建 Helm 客户端（第 234 行）效率较低，建议在循环外创建一次。

 func (hr *HelmReleaseController) BatchUninstallRelease(c *gin.Context) {
     var req struct {
         Names      []string `json:"name_list"`
         Namespaces []string `json:"ns_list"`
     }
     if err := c.ShouldBindJSON(&req); err != nil {
         amis.WriteJsonError(c, err)
         return
     }
+    
+    h, err := getHelm(c)
+    if err != nil {
+        amis.WriteJsonError(c, err)
+        return
+    }
+    
     for i := 0; i < len(req.Names); i++ {
         name := req.Names[i]
         ns := req.Namespaces[i]
-        h, err := getHelm(c)
-        if err != nil {
-            amis.WriteJsonError(c, err)
-            return
-        }

         // 检查权限
         _, _, err = handleCommonLogic(c, "delete", name, ns, "")

---

38-43: 错误变量重复声明可能造成混淆

多处使用 := 重新声明 err 变量，虽然语法正确，但可能影响代码可读性。

考虑在需要时使用不同的变量名或显式赋值以提高代码清晰度。

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