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

deiga · deiga · commit eef87e9db922 · 2026-01-27T22:50:31.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
@@ -686,7 +686,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
@@ -970,7 +970,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
`@@ -686,7 +686,7 @@ func resourceGithubRepositoryCreate(ctx context.Context, d *schema.ResourceData,`
`686`	`686`	`return diag.FromErr(err)`
`687`	`687`	`}`
`688`	`688`
`689`		`- d.SetId(*repo.Name)`
	`689`	`+ d.SetId(repo.GetName())`
`690`	`690`	`}`
`691`	`691`	`} else if d.Get("fork").(string) == "true" {`
`692`	`692`	`// Handle repository forking`
`@@ -970,7 +970,7 @@ func resourceGithubRepositoryUpdate(ctx context.Context, d *schema.ResourceData,`
`970`	`970`	`if err != nil {`
`971`	`971`	`return diag.FromErr(err)`
`972`	`972`	`}`
`973`		`- d.SetId(*repo.Name)`
	`973`	+ d.SetId(repo.GetName()) // It's possible that `repo.GetName()` is different from `repoName` if the repository is renamed
`974`	`974`
`975`	`975`	`if d.HasChange("pages") && !d.IsNewResource() {`
`976`	`976`	`opts := expandPagesUpdate(d.Get("pages").([]any))`