Add client nil checks to avoid panics (#2008)

spinillos · web-flow · commit f572d445ca96 · 2025-01-29T12:04:33.000+01:00
* Add client nil checks to avoid panics

* Update docs

* Fix docs again

* Update docs
diff --git a/docs/resources/dashboard_public.md b/docs/resources/dashboard_public.md
@@ -5,7 +5,7 @@ subcategory: "Grafana OSS"
 description: |-
   Manages Grafana public dashboards.
   Note: This resource is available only with Grafana 10.2+.
-  Official documentation https://grafana.com/docs/grafana/latest/dashboards/dashboard-public/HTTP API https://grafana.com/docs/grafana/next/developers/http_api/dashboard_public/
+  Official documentation https://grafana.com/docs/grafana/latest/dashboards/share-dashboards-panels/shared-dashboards/HTTP API https://grafana.com/docs/grafana/next/developers/http_api/dashboard_public/
 ---
 
 # grafana_dashboard_public (Resource)
@@ -14,7 +14,7 @@ Manages Grafana public dashboards.
 
 **Note:** This resource is available only with Grafana 10.2+.
 
-* [Official documentation](https://grafana.com/docs/grafana/latest/dashboards/dashboard-public/)
+* [Official documentation](https://grafana.com/docs/grafana/latest/dashboards/share-dashboards-panels/shared-dashboards/)
 * [HTTP API](https://grafana.com/docs/grafana/next/developers/http_api/dashboard_public/)
 
 ## Example Usage
diff --git a/internal/resources/cloud/common.go b/internal/resources/cloud/common.go
@@ -104,5 +104,14 @@ func (r *basePluginFrameworkResource) Configure(ctx context.Context, req resourc
 		return
 	}
 
+	if client.GrafanaCloudAPI == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the Grafana Cloud API.",
+			"Please ensure that cloud_api_url and cloud_access_policy_token are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.client = client.GrafanaCloudAPI
 }
diff --git a/internal/resources/grafana/common_plugin_framework.go b/internal/resources/grafana/common_plugin_framework.go
@@ -38,6 +38,15 @@ func (r *basePluginFrameworkDataSource) Configure(ctx context.Context, req datas
 		return
 	}
 
+	if client.GrafanaAPI == nil || client.GrafanaAPIConfig == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the Grafana API.",
+			"Please ensure that url and auth are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.client = client.GrafanaAPI
 	r.config = client.GrafanaAPIConfig
 }
@@ -81,6 +90,15 @@ func (r *basePluginFrameworkResource) Configure(ctx context.Context, req resourc
 		return
 	}
 
+	if client.GrafanaAPI == nil || client.GrafanaAPIConfig == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the Grafana API.",
+			"Please ensure that url and auth are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.client = client.GrafanaAPI
 	r.config = client.GrafanaAPIConfig
 }
diff --git a/internal/resources/grafana/resource_dashboard_public.go b/internal/resources/grafana/resource_dashboard_public.go
@@ -27,7 +27,7 @@ Manages Grafana public dashboards.
 
 **Note:** This resource is available only with Grafana 10.2+.
 
-* [Official documentation](https://grafana.com/docs/grafana/latest/dashboards/dashboard-public/)
+* [Official documentation](https://grafana.com/docs/grafana/latest/dashboards/share-dashboards-panels/shared-dashboards/)
 * [HTTP API](https://grafana.com/docs/grafana/next/developers/http_api/dashboard_public/)
 `,
 
diff --git a/internal/resources/machinelearning/resource_alert.go b/internal/resources/machinelearning/resource_alert.go
@@ -76,6 +76,15 @@ func (r *alertResource) Configure(ctx context.Context, req resource.ConfigureReq
 		return
 	}
 
+	if client.MLAPI == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the MLAPI API.",
+			"Please ensure that url and auth are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.mlapi = client.MLAPI
 }
 
diff --git a/internal/resources/oncall/resources.go b/internal/resources/oncall/resources.go
@@ -36,6 +36,15 @@ func (r *basePluginFrameworkResource) Configure(ctx context.Context, req resourc
 		return
 	}
 
+	if client.OnCallClient == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the OnCall API.",
+			"Please ensure that oncall_url and oncall_access_token/auth are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.client = client.OnCallClient
 }
 
@@ -60,6 +69,15 @@ func (r *basePluginFrameworkDataSource) Configure(ctx context.Context, req datas
 		return
 	}
 
+	if client.OnCallClient == nil {
+		resp.Diagnostics.AddError(
+			"The Grafana Provider is missing a configuration for the OnCall API.",
+			"Please ensure that oncall_url and oncall_access_token/auth are set in the provider configuration.",
+		)
+
+		return
+	}
+
 	r.client = client.OnCallClient
 }
 

Original file line number	Diff line number	Diff line change
`@@ -104,5 +104,14 @@ func (r *basePluginFrameworkResource) Configure(ctx context.Context, req resourc`
`104`	`104`	`return`
`105`	`105`	`}`
`106`	`106`
	`107`	`+ if client.GrafanaCloudAPI == nil {`
	`108`	`+ resp.Diagnostics.AddError(`
	`109`	`+ "The Grafana Provider is missing a configuration for the Grafana Cloud API.",`
	`110`	`+ "Please ensure that cloud_api_url and cloud_access_policy_token are set in the provider configuration.",`
	`111`	`+ )`
	`112`	`+`
	`113`	`+ return`
	`114`	`+ }`
	`115`	`+`
`107`	`116`	`r.client = client.GrafanaCloudAPI`
`108`	`117`	`}`
Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,15 @@ func (r *alertResource) Configure(ctx context.Context, req resource.ConfigureReq`
`76`	`76`	`return`
`77`	`77`	`}`
`78`	`78`
	`79`	`+ if client.MLAPI == nil {`
	`80`	`+ resp.Diagnostics.AddError(`
	`81`	`+ "The Grafana Provider is missing a configuration for the MLAPI API.",`
	`82`	`+ "Please ensure that url and auth are set in the provider configuration.",`
	`83`	`+ )`
	`84`	`+`
	`85`	`+ return`
	`86`	`+ }`
	`87`	`+`
`79`	`88`	`r.mlapi = client.MLAPI`
`80`	`89`	`}`
`81`	`90`