Refactor the git clone (#5190)

Move the authArgs into the singleflight closure, as they are only used within it. Signed-off-by: Shinnosuke Sawada-Dazai <[email protected]> Signed-off-by: pipecd-bot <[email protected]>
pipe-cd · Sep 5, 2024 · d27cff8 · d27cff8
1 parent 9910184
commit d27cff8
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/pkg/git/client.go b/pkg/git/client.go
@@ -145,15 +145,15 @@ func (c *client) Clone(ctx context.Context, repoID, remote, branch, destination
 		)
 	)
 
-	authArgs := []string{}
-	if c.username != "" && c.password != "" {
-		token := fmt.Sprintf("%s:%s", c.username, c.password)
-		encodedToken := base64.StdEncoding.EncodeToString([]byte(token))
-		header := fmt.Sprintf("Authorization: Basic %s", encodedToken)
-		authArgs = append(authArgs, "-c", fmt.Sprintf("http.extraHeader=%s", header))
-	}
-
 	_, err, _ := c.repoSingleFlights.Do(repoID, func() (interface{}, error) {
+		authArgs := []string{}
+		if c.username != "" && c.password != "" {
+			token := fmt.Sprintf("%s:%s", c.username, c.password)
+			encodedToken := base64.StdEncoding.EncodeToString([]byte(token))
+			header := fmt.Sprintf("Authorization: Basic %s", encodedToken)
+			authArgs = append(authArgs, "-c", fmt.Sprintf("http.extraHeader=%s", header))
+		}
+
 		_, err := os.Stat(repoCachePath)
 		if err != nil && !os.IsNotExist(err) {
 			return nil, err