Skip to content

Commit 7b690f1

Browse files
committed
refactor: Simplify user agent name handling and remove unnecessary string manipulation
1 parent 29a4e0f commit 7b690f1

File tree

4 files changed

+8
-6
lines changed

4 files changed

+8
-6
lines changed

internal/config/resource_base.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ func (r *RSCommon) SetClient(client *MongoDBClient) {
150150
func (r *RSCommon) asUserAgentExtra(ctx context.Context, reqOperation string, reqProviderMeta tfsdk.Config) UserAgentExtra {
151151
var meta ProviderMeta
152152
uaExtra := UserAgentExtra{
153-
Name: r.ResourceName,
153+
Name: userAgentNameValue(r.ResourceName),
154154
Operation: reqOperation,
155155
}
156156
if reqProviderMeta.Raw.IsNull() {

internal/config/resource_base_sdkv2.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ func (a *AnalyticsResourceSDKv2) updateContextWithProviderMeta(ctx context.Conte
178178
}
179179

180180
uaExtra := UserAgentExtra{
181-
Name: a.name,
181+
Name: userAgentNameValue(a.name),
182182
Operation: operationName,
183183
Extras: meta.UserAgentExtra,
184184
ModuleName: moduleName,

internal/config/user_agent.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,10 @@ type UserAgentExtra struct {
3333
ModuleVersion string
3434
}
3535

36+
func userAgentNameValue(name string) string {
37+
return strings.TrimPrefix(name, "mongodbatlas_")
38+
}
39+
3640
// Combine returns a new UserAgentExtra by merging the receiver with another.
3741
// Non-empty fields in 'other' take precedence over the receiver's fields.
3842
func (e UserAgentExtra) Combine(other UserAgentExtra) UserAgentExtra {

internal/provider/provider_sdk2.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package provider
33
import (
44
"context"
55
"fmt"
6-
"strings"
76

87
"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
98
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
@@ -292,9 +291,8 @@ func getResourcesMap() map[string]*schema.Resource {
292291
"mongodbatlas_cluster_outage_simulation": clusteroutagesimulation.Resource(),
293292
}
294293
analyticsMap := map[string]*schema.Resource{}
295-
for fullName, resource := range resourcesMap {
296-
name := strings.TrimPrefix(fullName, "mongodbatlas_")
297-
analyticsMap[fullName] = config.NewAnalyticsResourceSDKv2(resource, name)
294+
for name, resource := range resourcesMap {
295+
analyticsMap[name] = config.NewAnalyticsResourceSDKv2(resource, name)
298296
}
299297
return analyticsMap
300298
}

0 commit comments

Comments
 (0)