Skip to content

Commit

Permalink
Removed the “SetId” as it was unnecessary
Browse files Browse the repository at this point in the history
  • Loading branch information
wchrisjohnson committed May 6, 2017
1 parent da375cd commit 00d0061
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions builtin/providers/heroku/resource_heroku_cert.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,14 +98,10 @@ func resourceHerokuCertUpdate(d *schema.ResourceData, meta interface{}) error {

if d.HasChange("certificate_chain") || d.HasChange("private_key") {
log.Printf("[DEBUG] SSL Certificate update configuration: %#v, %#v", app, opts)
ad, err := client.SSLEndpointUpdate(context.TODO(), app, d.Id(), opts)
_, err := client.SSLEndpointUpdate(context.TODO(), app, d.Id(), opts)
if err != nil {
return fmt.Errorf("Error updating SSL endpoint: %s", err)
}

// Store the new ID
d.SetId(ad.ID)
log.Printf("[INFO] Updated SSL Certificate ID: %s", d.Id())
}

return resourceHerokuCertRead(d, meta)
Expand Down

0 comments on commit 00d0061

Please sign in to comment.