Skip to content

Commit 16e971f

Browse files
committed
fix(api/models): PAM ProviderType models Name to use string rather than pointer
1 parent 3580c64 commit 16e971f

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

v3/api/pam_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ func TestCreatePAMProvider(t *testing.T) {
190190
Area: 1,
191191
ProviderType: ProviderType{
192192
Id: mockProviderTypeId,
193-
Name: &mockProviderTypeName,
193+
Name: mockProviderTypeName,
194194
},
195195
SecuredAreaId: intPtr(1),
196196
}
@@ -279,7 +279,7 @@ func TestUpdatePAMProvider(t *testing.T) {
279279
Area: 1,
280280
ProviderType: ProviderType{
281281
Id: mockProviderTypeId,
282-
Name: &mockProviderTypeName,
282+
Name: mockProviderTypeName,
283283
},
284284
SecuredAreaId: intPtr(1),
285285
}
@@ -290,7 +290,7 @@ func TestUpdatePAMProvider(t *testing.T) {
290290
Area: 1,
291291
ProviderType: &ProviderType{
292292
Id: mockProviderTypeId,
293-
Name: &mockProviderTypeName,
293+
Name: mockProviderTypeName,
294294
},
295295
SecuredAreaId: intPtr(1),
296296
Remote: false,

v3/api/pam_types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ func (c *Client) GetPAMProviderTypeByName(name string) (*ProviderTypeResponse, e
6060

6161
// find the provider type with the matching name
6262
for _, t := range *types {
63-
if t.Name != nil && *t.Name == name {
63+
if t.Name == name {
6464
return &t, nil
6565
}
6666
}

v3/api/pam_types_models.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ type SecretType int
3030
// ProviderTypeParameterResponse represents a parameter for a PAM provider type
3131
type ProviderTypeParameterResponse struct {
3232
Id int `json:"Id,omitempty"`
33-
Name *string `json:"Name,omitempty"`
33+
Name string `json:"Name,omitempty"`
3434
DisplayName *string `json:"DisplayName,omitempty"`
3535
DataType PamParameterDataType `json:"DataType,omitempty"`
3636
InstanceLevel bool `json:"InstanceLevel,omitempty"`
@@ -39,7 +39,7 @@ type ProviderTypeParameterResponse struct {
3939
// ProviderTypeResponse represents a PAM provider type
4040
type ProviderTypeResponse struct {
4141
Id string `json:"Id,omitempty"` // UUID format
42-
Name *string `json:"Name,omitempty"`
42+
Name string `json:"Name,omitempty"`
4343
Parameters *[]ProviderTypeParameterResponse `json:"Parameters,omitempty"`
4444
}
4545

@@ -60,15 +60,15 @@ type ProviderTypeCreateRequest struct {
6060
// ProviderCreateRequestProviderTypeParam represents a provider type parameter in a provider creation request
6161
type ProviderCreateRequestProviderTypeParam struct {
6262
Id int `json:"Id,omitempty"`
63-
Name *string `json:"Name,omitempty"`
63+
Name string `json:"Name,omitempty"`
6464
DisplayName *string `json:"DisplayName,omitempty"`
6565
InstanceLevel bool `json:"InstanceLevel,omitempty"`
6666
}
6767

6868
// PamProviderTypeParam represents a provider type parameter (full model) for PAM operations
6969
type PamProviderTypeParam struct {
7070
Id int `json:"Id,omitempty"`
71-
Name *string `json:"Name,omitempty"`
71+
Name string `json:"Name,omitempty"`
7272
DisplayName *string `json:"DisplayName,omitempty"`
7373
DataType PamParameterDataType `json:"DataType,omitempty"`
7474
InstanceLevel bool `json:"InstanceLevel,omitempty"`
@@ -78,7 +78,7 @@ type PamProviderTypeParam struct {
7878
// ProviderType represents a PAM provider type (full model)
7979
type ProviderType struct {
8080
Id string `json:"Id,omitempty"` // UUID format
81-
Name *string `json:"Name,omitempty"`
81+
Name string `json:"Name,omitempty"`
8282
ProviderTypeParams *[]PamProviderTypeParam `json:"ProviderTypeParams,omitempty"`
8383
}
8484

v3/api/pam_types_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ var (
8787
Area: 1,
8888
ProviderType: &ProviderType{
8989
Id: mockProviderTypeId,
90-
Name: &mockProviderTypeName,
90+
Name: mockProviderTypeName,
9191
},
9292
SecuredAreaId: intPtr(1),
9393
Remote: false,

0 commit comments

Comments
 (0)