diff options
| author | Lunny Xiao <xiaolunwen@gmail.com> | 2025-03-15 19:48:59 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-15 19:48:59 -0700 |
| commit | 0056fdb94201f54fcbb51d741a68b04bf41213fc (patch) | |
| tree | 5185b6ad1d8c8560e8cb06cb83a60d6c8fc35620 /services/automerge | |
| parent | f11ac6bf3cb45e01080b1ec5bd9cbdd1ee5cda92 (diff) | |
| download | gitea-0056fdb94201f54fcbb51d741a68b04bf41213fc.tar.gz gitea-0056fdb94201f54fcbb51d741a68b04bf41213fc.zip | |
Move git references checking to gitrepo packages to reduce expose of repository path (#33891)
Diffstat (limited to 'services/automerge')
| -rw-r--r-- | services/automerge/automerge.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/services/automerge/automerge.go b/services/automerge/automerge.go index bdb0493ae8..62d560ff94 100644 --- a/services/automerge/automerge.go +++ b/services/automerge/automerge.go @@ -248,13 +248,13 @@ func handlePullRequestAutoMerge(pullID int64, sha string) { switch pr.Flow { case issues_model.PullRequestFlowGithub: - headBranchExist := headGitRepo.IsBranchExist(pr.HeadBranch) - if pr.HeadRepo == nil || !headBranchExist { + headBranchExist := pr.HeadRepo != nil && gitrepo.IsBranchExist(ctx, pr.HeadRepo, pr.HeadBranch) + if !headBranchExist { log.Warn("Head branch of auto merge %-v does not exist [HeadRepoID: %d, Branch: %s]", pr, pr.HeadRepoID, pr.HeadBranch) return } case issues_model.PullRequestFlowAGit: - headBranchExist := git.IsReferenceExist(ctx, baseGitRepo.Path, pr.GetGitRefName()) + headBranchExist := gitrepo.IsReferenceExist(ctx, pr.BaseRepo, pr.GetGitRefName()) if !headBranchExist { log.Warn("Head branch of auto merge %-v does not exist [HeadRepoID: %d, Branch(Agit): %s]", pr, pr.HeadRepoID, pr.HeadBranch) return |