cr: add check for empty password

gkech · gkech · commit 076383db058f · 2026-07-06T16:15:59.000+03:00
diff --git a/pkg/controller/perconaservermongodbclustersync/target_user.go b/pkg/controller/perconaservermongodbclustersync/target_user.go
@@ -100,10 +100,10 @@ func (r *ReconcilePerconaServerMongoDBClusterSync) ensureTargetUserSecret(
 	err := r.client.Get(ctx, nn, existing)
 	switch {
 	case err == nil:
-		if _, ok := existing.Data[targetUserSecretUsernameKey]; !ok {
+		if v, ok := existing.Data[targetUserSecretUsernameKey]; !ok || len(v) == 0 {
 			return nil, errors.Errorf("target user secret %s missing %s key", nn, targetUserSecretUsernameKey)
 		}
-		if _, ok := existing.Data[targetUserSecretPasswordKey]; !ok {
+		if v, ok := existing.Data[targetUserSecretPasswordKey]; !ok || len(v) == 0 {
 			return nil, errors.Errorf("target user secret %s missing %s key", nn, targetUserSecretPasswordKey)
 		}
 		return existing, nil

Original file line number	Diff line number	Diff line change
`@@ -100,10 +100,10 @@ func (r *ReconcilePerconaServerMongoDBClusterSync) ensureTargetUserSecret(`
`100`	`100`	`err := r.client.Get(ctx, nn, existing)`
`101`	`101`	`switch {`
`102`	`102`	`case err == nil:`
`103`		`- if _, ok := existing.Data[targetUserSecretUsernameKey]; !ok {`
	`103`	`+ if v, ok := existing.Data[targetUserSecretUsernameKey]; !ok \|\| len(v) == 0 {`
`104`	`104`	`return nil, errors.Errorf("target user secret %s missing %s key", nn, targetUserSecretUsernameKey)`
`105`	`105`	`}`
`106`		`- if _, ok := existing.Data[targetUserSecretPasswordKey]; !ok {`
	`106`	`+ if v, ok := existing.Data[targetUserSecretPasswordKey]; !ok \|\| len(v) == 0 {`
`107`	`107`	`return nil, errors.Errorf("target user secret %s missing %s key", nn, targetUserSecretPasswordKey)`
`108`	`108`	`}`
`109`	`109`	`return existing, nil`