Add an api endpoint to fetch git notes (#15373) (#16649)

close #15373

1 files changed, 12 insertions, 1 deletions

diff --git a/modules/git/notes_nogogit.go b/modules/git/notes_nogogit.go
index 267087a86f..697f998288 100644
--- a/modules/git/notes_nogogit.go
+++ b/modules/git/notes_nogogit.go
@@ -10,20 +10,26 @@ import (
 	"context"
 	"io/ioutil"
 	"strings"
+
+	"code.gitea.io/gitea/modules/log"
 )
 
 // GetNote retrieves the git-notes data for a given commit.
 func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note) error {
+	log.Trace("Searching for git note corresponding to the commit %q in the repository %q", commitID, repo.Path)
 	notes, err := repo.GetCommit(NotesRef)
 	if err != nil {
+		log.Error("Unable to get commit from ref %q. Error: %v", NotesRef, err)
 		return err
 	}
 
 	path := ""
 
 	tree := &notes.Tree
+	log.Trace("Found tree with ID %q while searching for git note corresponding to the commit %q", tree.ID, commitID)
 
 	var entry *TreeEntry
+	originalCommitID := commitID
 	for len(commitID) > 2 {
 		entry, err = tree.GetTreeEntryByPath(commitID)
 		if err == nil {
@@ -36,12 +42,15 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note)
 			commitID = commitID[2:]
 		}
 		if err != nil {
+			log.Error("Unable to find git note corresponding to the commit %q. Error: %v", originalCommitID, err)
 			return err
 		}
 	}
 
-	dataRc, err := entry.Blob().DataAsync()
+	blob := entry.Blob()
+	dataRc, err := blob.DataAsync()
 	if err != nil {
+		log.Error("Unable to read blob with ID %q. Error: %v", blob.ID, err)
 		return err
 	}
 	closed := false
@@ -52,6 +61,7 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note)
 	}()
 	d, err := ioutil.ReadAll(dataRc)
 	if err != nil {
+		log.Error("Unable to read blob with ID %q. Error: %v", blob.ID, err)
 		return err
 	}
 	_ = dataRc.Close()
@@ -66,6 +76,7 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note)
 
 	lastCommits, err := GetLastCommitForPaths(ctx, notes, treePath, []string{path})
 	if err != nil {
+		log.Error("Unable to get the commit for the path %q. Error: %v", treePath, err)
 		return err
 	}
 	note.Commit = lastCommits[path]