You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by ly...@apache.org on 2024/03/16 04:53:27 UTC

(incubator-devlake) 01/01: fix(gitextractor): update log, print raw error detail

This is an automated email from the ASF dual-hosted git repository.

lynwee pushed a commit to branch lw-0316
in repository https://gitbox.apache.org/repos/asf/incubator-devlake.git

commit 297e00d68d8f6e0f6ecbfc3c741b51e5ecde7823
Author: houlinwei <ti...@gmail.com>
AuthorDate: Sat Mar 16 12:53:14 2024 +0800

    fix(gitextractor): update log, print raw error detail
---
 backend/plugins/gitextractor/parser/clone.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/plugins/gitextractor/parser/clone.go b/backend/plugins/gitextractor/parser/clone.go
index 809aeaafd..7b5fc1ff9 100644
--- a/backend/plugins/gitextractor/parser/clone.go
+++ b/backend/plugins/gitextractor/parser/clone.go
@@ -121,13 +121,13 @@ func (l *GitRepoCreator) cloneOverHTTP(ctx plugin.SubTaskContext, withGoGit bool
 		_, err := gogit.PlainCloneContext(ctx.GetContext(), dir, true, cloneOptions)
 		done <- struct{}{}
 		if err != nil {
+			l.logger.Error(err, "PlainCloneContext")
 			// Some sensitive information such as password will be released in this err.
 			if err.Error() == "repository not found" {
 				// do nothing, it's a safe error message.
 			} else {
 				err = fmt.Errorf("plain clone git error")
 			}
-			l.logger.Error(err, "PlainCloneContext")
 			return nil, err
 		}
 		if withGoGit {