Explorar el Código

database: use safe git-module API for tag deletion (#8175)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ᴊᴏᴇ ᴄʜᴇɴ hace 2 días
padre
commit
a000f0c7a6
Se han modificado 1 ficheros con 7 adiciones y 6 borrados
  1. 7 6
      internal/database/release.go

+ 7 - 6
internal/database/release.go

@@ -13,7 +13,6 @@ import (
 	"github.com/gogs/git-module"
 	"github.com/gogs/git-module"
 
 
 	"gogs.io/gogs/internal/errutil"
 	"gogs.io/gogs/internal/errutil"
-	"gogs.io/gogs/internal/process"
 	apiv1types "gogs.io/gogs/internal/route/api/v1/types"
 	apiv1types "gogs.io/gogs/internal/route/api/v1/types"
 )
 )
 
 
@@ -359,11 +358,13 @@ func DeleteReleaseOfRepoByID(repoID, id int64) error {
 		return errors.Newf("GetRepositoryByID: %v", err)
 		return errors.Newf("GetRepositoryByID: %v", err)
 	}
 	}
 
 
-	_, stderr, err := process.ExecDir(-1, repo.RepoPath(),
-		fmt.Sprintf("DeleteReleaseByID (git tag -d): %d", rel.ID),
-		"git", "tag", "-d", rel.TagName)
-	if err != nil && !strings.Contains(stderr, "not found") {
-		return errors.Newf("git tag -d: %v - %s", err, stderr)
+	gitRepo, err := git.Open(repo.RepoPath())
+	if err != nil {
+		return errors.Newf("open repository: %v", err)
+	}
+	err = gitRepo.DeleteTag(rel.TagName)
+	if err != nil && !strings.Contains(err.Error(), "not found") {
+		return errors.Newf("delete tag: %v", err)
 	}
 	}
 
 
 	if _, err = x.Id(rel.ID).Delete(new(Release)); err != nil {
 	if _, err = x.Id(rel.ID).Delete(new(Release)); err != nil {