fix(dbaas): access control fields to pointers (#292)

Author: villevsv-upcloud

CHANGELOG.md

@@ -5,6 +5,11 @@ See updating [Changelog example here](https://keepachangelog.com/en/1.0.0/)
 
 ## [Unreleased]
 
+### Changed
+- **Breaking**, Managed Database: `ManagedDatabaseUserOpernSearchAccessControl` fields changed to pointers
+- **Breaking**, Managed Database: `ManagedDatabaseUserPGAccessControl` fields changed to pointers
+- **Breaking**, Managed Database: `ManagedDatabaseUserRedisAccessControl` fields changed to pointers
+
 ### Removed
 - **Breaking**, Managed Database: connection related methods in favor of session
 

upcloud/managed_database.go

@@ -618,18 +618,18 @@ type ManagedDatabaseUser struct {
 }
 
 type ManagedDatabaseUserPGAccessControl struct {
-	AllowReplication bool `json:"allow_replication"`
+	AllowReplication *bool `json:"allow_replication,omitempty"`
 }
 
 type ManagedDatabaseUserRedisAccessControl struct {
-	Categories []string `json:"categories,omitempty"`
-	Channels   []string `json:"channels,omitempty"`
-	Commands   []string `json:"commands,omitempty"`
-	Keys       []string `json:"keys,omitempty"`
+	Categories *[]string `json:"categories,omitempty"`
+	Channels   *[]string `json:"channels,omitempty"`
+	Commands   *[]string `json:"commands,omitempty"`
+	Keys       *[]string `json:"keys,omitempty"`
 }
 
 type ManagedDatabaseUserOpenSearchAccessControl struct {
-	Rules []ManagedDatabaseUserOpenSearchAccessControlRule `json:"rules"`
+	Rules *[]ManagedDatabaseUserOpenSearchAccessControlRule `json:"rules,omitempty"`
 }
 
 type ManagedDatabaseUserOpenSearchAccessControlRule struct {

upcloud/managed_database_test.go

@@ -776,13 +776,13 @@ func TestManagedDatabaseUser(t *testing.T) {
 		Password:       "new-password",
 		Username:       "api-doc-user",
 		PGAccessControl: &ManagedDatabaseUserPGAccessControl{
-			AllowReplication: false,
+			AllowReplication: BoolPtr(false),
 		},
 		RedisAccessControl: &ManagedDatabaseUserRedisAccessControl{
-			Categories: []string{"+@set"},
-			Channels:   []string{"*"},
-			Commands:   []string{"+set"},
-			Keys:       []string{"key_*"},
+			Categories: &[]string{"+@set"},
+			Channels:   &[]string{"*"},
+			Commands:   &[]string{"+set"},
+			Keys:       &[]string{"key_*"},
 		},
 	}
 	assert.Equal(t, want, got)

upcloud/request/managed_database_test.go

@@ -429,13 +429,13 @@ func TestCreateManagedDatabaseUserRequest(t *testing.T) {
 		Password:       "new-password",
 		Authentication: upcloud.ManagedDatabaseUserAuthenticationCachingSHA2Password,
 		PGAccessControl: &upcloud.ManagedDatabaseUserPGAccessControl{
-			AllowReplication: true,
+			AllowReplication: upcloud.BoolPtr(true),
 		},
 		RedisAccessControl: &upcloud.ManagedDatabaseUserRedisAccessControl{
-			Categories: []string{"+@set"},
-			Channels:   []string{"*"},
-			Commands:   []string{"+set"},
-			Keys:       []string{"key_*"},
+			Categories: &[]string{"+@set"},
+			Channels:   &[]string{"*"},
+			Commands:   &[]string{"+set"},
+			Keys:       &[]string{"key_*"},
 		},
 	}
 	got, err := json.Marshal(&r)
@@ -475,13 +475,13 @@ func TestModifyManagedDatabaseUserAccessControlRequest(t *testing.T) {
 		ServiceUUID: "fakeuuid",
 		Username:    "fakeuser",
 		PGAccessControl: &upcloud.ManagedDatabaseUserPGAccessControl{
-			AllowReplication: true,
+			AllowReplication: upcloud.BoolPtr(true),
 		},
 		RedisAccessControl: &upcloud.ManagedDatabaseUserRedisAccessControl{
-			Categories: []string{"+@set"},
-			Channels:   []string{"*"},
-			Commands:   []string{"+set"},
-			Keys:       []string{"key_*"},
+			Categories: &[]string{"+@set"},
+			Channels:   &[]string{"*"},
+			Commands:   &[]string{"+set"},
+			Keys:       &[]string{"key_*"},
 		},
 	}
 	got, err := json.Marshal(&r)