Use repo.GetName() instead of *repo.Name

deiga · deiga · commit f4349542eca9 · 2026-02-03T16:14:51.000+02:00
Signed-off-by: Timo Sand &lt;timo.sand@f-secure.com&gt;
diff --git a/github/resource_github_repository.go b/github/resource_github_repository.go
@@ -683,7 +683,7 @@ func resourceGithubRepositoryCreate(ctx context.Context, d *schema.ResourceData,
 				return diag.FromErr(err)
 			}
 
-			d.SetId(*repo.Name)
+			d.SetId(repo.GetName())
 		}
 	} else if d.Get("fork").(string) == "true" {
 		// Handle repository forking
@@ -963,7 +963,7 @@ func resourceGithubRepositoryUpdate(ctx context.Context, d *schema.ResourceData,
 	if err != nil {
 		return diag.FromErr(err)
 	}
-	d.SetId(*repo.Name)
+	d.SetId(repo.GetName()) // It's possible that `repo.GetName()` is different from `repoName` if the repository is renamed
 
 	if d.HasChange("pages") && !d.IsNewResource() {
 		opts := expandPagesUpdate(d.Get("pages").([]any))

Original file line number	Diff line number	Diff line change
`@@ -683,7 +683,7 @@ func resourceGithubRepositoryCreate(ctx context.Context, d *schema.ResourceData,`
`683`	`683`	`return diag.FromErr(err)`
`684`	`684`	`}`
`685`	`685`
`686`		`- d.SetId(*repo.Name)`
	`686`	`+ d.SetId(repo.GetName())`
`687`	`687`	`}`
`688`	`688`	`} else if d.Get("fork").(string) == "true" {`
`689`	`689`	`// Handle repository forking`
`@@ -963,7 +963,7 @@ func resourceGithubRepositoryUpdate(ctx context.Context, d *schema.ResourceData,`
`963`	`963`	`if err != nil {`
`964`	`964`	`return diag.FromErr(err)`
`965`	`965`	`}`
`966`		`- d.SetId(*repo.Name)`
	`966`	+ d.SetId(repo.GetName()) // It's possible that `repo.GetName()` is different from `repoName` if the repository is renamed
`967`	`967`
`968`	`968`	`if d.HasChange("pages") && !d.IsNewResource() {`
`969`	`969`	`opts := expandPagesUpdate(d.Get("pages").([]any))`