add context to CheckAtrribute

Author: TheFox0x7

modules/git/repo_attribute.go

@@ -24,7 +24,7 @@ type CheckAttributeOpts struct {
 }
 
 // CheckAttribute return the Blame object of file
-func (repo *Repository) CheckAttribute(opts CheckAttributeOpts) (map[string]map[string]string, error) {
+func (repo *Repository) CheckAttribute(ctx context.Context, opts CheckAttributeOpts) (map[string]map[string]string, error) {
 	env := []string{}
 
 	if len(opts.IndexFile) > 0 {
@@ -59,7 +59,7 @@ func (repo *Repository) CheckAttribute(opts CheckAttributeOpts) (map[string]map[
 
 	cmd.AddDashesAndList(opts.Filenames...)
 
-	if err := cmd.Run(repo.Ctx, &RunOpts{
+	if err := cmd.Run(ctx, &RunOpts{
 		Env:    env,
 		Dir:    repo.Path,
 		Stdout: stdOut,

modules/git/repo_language_stats_nogogit.go

@@ -21,7 +21,7 @@ import (
 func (repo *Repository) GetLanguageStats(ctx context.Context, commitID string) (map[string]int64, error) {
 	// We will feed the commit IDs in order into cat-file --batch, followed by blobs as necessary.
 	// so let's create a batch stdin and stdout
-	batchStdinWriter, batchReader, cancel, err := repo.CatFileBatch(repo.Ctx)
+	batchStdinWriter, batchReader, cancel, err := repo.CatFileBatch(ctx)
 	if err != nil {
 		return nil, err
 	}
@@ -80,8 +80,8 @@ func (repo *Repository) GetLanguageStats(ctx context.Context, commitID string) (
 
 	for _, f := range entries {
 		select {
-		case <-repo.Ctx.Done():
-			return sizes, repo.Ctx.Err()
+		case <-ctx.Done():
+			return sizes, ctx.Err()
 		default:
 		}
 

routers/web/repo/setting/lfs.go

@@ -150,7 +150,7 @@ func LFSLocks(ctx *context.Context) {
 		return
 	}
 
-	name2attribute2info, err := gitRepo.CheckAttribute(git.CheckAttributeOpts{
+	name2attribute2info, err := gitRepo.CheckAttribute(ctx, git.CheckAttributeOpts{
 		Attributes: []string{"lockable"},
 		Filenames:  filenames,
 		CachedOnly: true,

services/repository/files/content.go

@@ -279,7 +279,7 @@ func TryGetContentLanguage(ctx context.Context, gitRepo *git.Repository, commitI
 
 	defer deleteTemporaryFile()
 
-	filename2attribute2info, err := gitRepo.CheckAttribute(git.CheckAttributeOpts{
+	filename2attribute2info, err := gitRepo.CheckAttribute(ctx, git.CheckAttributeOpts{
 		CachedOnly: true,
 		Attributes: []string{git.AttributeLinguistLanguage, git.AttributeGitlabLanguage},
 		Filenames:  []string{treePath},

services/repository/files/update.go

@@ -483,7 +483,7 @@ func CreateOrUpdateFile(ctx context.Context, t *TemporaryUploadRepository, file
 	var lfsMetaObject *git_model.LFSMetaObject
 	if setting.LFS.StartServer && hasOldBranch {
 		// Check there is no way this can return multiple infos
-		filename2attribute2info, err := t.gitRepo.CheckAttribute(git.CheckAttributeOpts{
+		filename2attribute2info, err := t.gitRepo.CheckAttribute(ctx, git.CheckAttributeOpts{
 			Attributes: []string{"filter"},
 			Filenames:  []string{file.Options.treePath},
 			CachedOnly: true,

services/repository/files/upload.go

@@ -107,7 +107,7 @@ func UploadRepoFiles(ctx context.Context, repo *repo_model.Repository, doer *use
 
 	var filename2attribute2info map[string]map[string]string
 	if setting.LFS.StartServer {
-		filename2attribute2info, err = t.gitRepo.CheckAttribute(git.CheckAttributeOpts{
+		filename2attribute2info, err = t.gitRepo.CheckAttribute(ctx, git.CheckAttributeOpts{
 			Attributes: []string{"filter"},
 			Filenames:  names,
 			CachedOnly: true,