add context to branch commands

Author: TheFox0x7

modules/git/repo_branch.go

@@ -34,11 +34,11 @@ type Branch struct {
 }
 
 // GetHEADBranch returns corresponding branch of HEAD.
-func (repo *Repository) GetHEADBranch() (*Branch, error) {
+func (repo *Repository) GetHEADBranch(ctx context.Context) (*Branch, error) {
 	if repo == nil {
 		return nil, fmt.Errorf("nil repo")
 	}
-	stdout, _, err := NewCommand("symbolic-ref", "HEAD").RunStdString(repo.Ctx, &RunOpts{Dir: repo.Path})
+	stdout, _, err := NewCommand("symbolic-ref", "HEAD").RunStdString(ctx, &RunOpts{Dir: repo.Path})
 	if err != nil {
 		return nil, err
 	}
@@ -104,7 +104,7 @@ type DeleteBranchOptions struct {
 }
 
 // DeleteBranch delete a branch by name on repository.
-func (repo *Repository) DeleteBranch(name string, opts DeleteBranchOptions) error {
+func (repo *Repository) DeleteBranch(ctx context.Context, name string, opts DeleteBranchOptions) error {
 	cmd := NewCommand("branch")
 
 	if opts.Force {
@@ -114,17 +114,17 @@ func (repo *Repository) DeleteBranch(name string, opts DeleteBranchOptions) erro
 	}
 
 	cmd.AddDashesAndList(name)
-	_, _, err := cmd.RunStdString(repo.Ctx, &RunOpts{Dir: repo.Path})
+	_, _, err := cmd.RunStdString(ctx, &RunOpts{Dir: repo.Path})
 
 	return err
 }
 
 // CreateBranch create a new branch
-func (repo *Repository) CreateBranch(branch, oldbranchOrCommit string) error {
+func (repo *Repository) CreateBranch(ctx context.Context, branch, oldbranchOrCommit string) error {
 	cmd := NewCommand("branch")
 	cmd.AddDashesAndList(branch, oldbranchOrCommit)
 
-	_, _, err := cmd.RunStdString(repo.Ctx, &RunOpts{Dir: repo.Path})
+	_, _, err := cmd.RunStdString(ctx, &RunOpts{Dir: repo.Path})
 
 	return err
 }

routers/api/v1/repo/commits.go

@@ -180,7 +180,7 @@ func GetAllCommits(ctx *context.APIContext) {
 		var baseCommit *git.Commit
 		if len(sha) == 0 {
 			// no sha supplied - use default branch
-			head, err := ctx.Repo.GitRepo.GetHEADBranch()
+			head, err := ctx.Repo.GitRepo.GetHEADBranch(ctx)
 			if err != nil {
 				ctx.APIErrorInternal(err)
 				return

services/repository/branch.go

@@ -550,7 +550,7 @@ func DeleteBranch(ctx context.Context, doer *user_model.User, repo *repo_model.R
 			}
 		}
 
-		return gitRepo.DeleteBranch(branchName, git.DeleteBranchOptions{
+		return gitRepo.DeleteBranch(ctx, branchName, git.DeleteBranchOptions{
 			Force: true,
 		})
 	}); err != nil {

services/repository/migrate.go

@@ -140,7 +140,7 @@ func MigrateRepositoryGitData(ctx context.Context, u *user_model.User,
 	if !repo.IsEmpty {
 		if len(repo.DefaultBranch) == 0 {
 			// Try to get HEAD branch and set it as default branch.
-			headBranch, err := gitRepo.GetHEADBranch()
+			headBranch, err := gitRepo.GetHEADBranch(ctx)
 			if err != nil {
 				return repo, fmt.Errorf("GetHEADBranch: %w", err)
 			}