Skip to content

Commit 6bb5f80

Browse files
committed
fix(provider): set User-Agent on clients that were missing it
The Grafana OpenAPI, SLO, and CloudProvider clients were not setting the User-Agent header, making it impossible to identify the source of API requests in server logs.
1 parent 6b19029 commit 6bb5f80

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

pkg/provider/configure_clients.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,7 @@ func createGrafanaAPIClient(client *common.Client, providerConfig ProviderConfig
173173
if cfg.HTTPHeaders, err = getHTTPHeadersMap(providerConfig); err != nil {
174174
return err
175175
}
176+
cfg.HTTPHeaders["User-Agent"] = providerConfig.UserAgent.ValueString()
176177
client.GrafanaAPI = goapi.NewHTTPClientWithConfig(strfmt.Default, &cfg)
177178
client.GrafanaAPIConfig = &cfg
178179

@@ -269,6 +270,7 @@ func createSLOClient(client *common.Client, providerConfig ProviderConfig) error
269270
sloConfig.Scheme = client.GrafanaAPIURLParsed.Scheme
270271
sloConfig.DefaultHeader, err = getHTTPHeadersMap(providerConfig)
271272
sloConfig.DefaultHeader["Authorization"] = "Bearer " + providerConfig.Auth.ValueString()
273+
sloConfig.UserAgent = providerConfig.UserAgent.ValueString()
272274
sloConfig.HTTPClient = getRetryClient(providerConfig)
273275
client.SLOClient = slo.NewAPIClient(sloConfig)
274276

@@ -310,6 +312,7 @@ func createCloudProviderClient(client *common.Client, providerConfig ProviderCon
310312
if err != nil {
311313
return fmt.Errorf("failed to get provider default HTTP headers: %w", err)
312314
}
315+
providerHeaders["User-Agent"] = providerConfig.UserAgent.ValueString()
313316

314317
apiClient, err := cloudproviderapi.NewClient(
315318
providerConfig.CloudProviderAccessToken.ValueString(),

0 commit comments

Comments
 (0)