fix: skip app functionality when app isnt configured

go-vela · Dec 18, 2024 · 5a39382 · 5a39382
1 parent f9f0378
commit 5a39382
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/scm/github/repo.go b/scm/github/repo.go
@@ -685,6 +685,11 @@ func (c *client) GetNetrcPassword(ctx context.Context, db database.Interface, r
 
 	l.Tracef("getting netrc password for %s/%s", r.GetOrg(), r.GetName())
 
+	// no GitHub App configured, use legacy oauth token
+	if c.AppsTransport == nil {
+		return u.GetToken(), nil
+	}
+
 	var err error
 
 	// repos that the token has access to
@@ -770,6 +775,11 @@ func (c *client) SyncRepoWithInstallation(ctx context.Context, r *api.Repo) (*ap
 		"repo": r.GetName(),
 	}).Tracef("syncing app installation for repo %s/%s", r.GetOrg(), r.GetName())
 
+	// no GitHub App configured, skip
+	if c.AppsTransport == nil {
+		return r, nil
+	}
+
 	client, err := c.newGithubAppClient()
 	if err != nil {
 		return r, err