Skip to content

Commit 81b3be5

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 c1d1c33 commit 81b3be5

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
@@ -172,6 +172,7 @@ func createGrafanaAPIClient(client *common.Client, providerConfig ProviderConfig
172172
if cfg.HTTPHeaders, err = getHTTPHeadersMap(providerConfig); err != nil {
173173
return err
174174
}
175+
cfg.HTTPHeaders["User-Agent"] = providerConfig.UserAgent.ValueString()
175176
client.GrafanaAPI = goapi.NewHTTPClientWithConfig(strfmt.Default, &cfg)
176177
client.GrafanaAPIConfig = &cfg
177178

@@ -266,6 +267,7 @@ func createSLOClient(client *common.Client, providerConfig ProviderConfig) error
266267
sloConfig.Scheme = client.GrafanaAPIURLParsed.Scheme
267268
sloConfig.DefaultHeader, err = getHTTPHeadersMap(providerConfig)
268269
sloConfig.DefaultHeader["Authorization"] = "Bearer " + providerConfig.Auth.ValueString()
270+
sloConfig.UserAgent = providerConfig.UserAgent.ValueString()
269271
sloConfig.HTTPClient = getRetryClient(providerConfig)
270272
client.SLOClient = slo.NewAPIClient(sloConfig)
271273

@@ -307,6 +309,7 @@ func createCloudProviderClient(client *common.Client, providerConfig ProviderCon
307309
if err != nil {
308310
return fmt.Errorf("failed to get provider default HTTP headers: %w", err)
309311
}
312+
providerHeaders["User-Agent"] = providerConfig.UserAgent.ValueString()
310313

311314
apiClient, err := cloudproviderapi.NewClient(
312315
providerConfig.CloudProviderAccessToken.ValueString(),

0 commit comments

Comments
 (0)