From d7d825a5e6c1543d66cc29cfdbe981c2c1c502f8 Mon Sep 17 00:00:00 2001 From: Takuya Wakazono Date: Mon, 2 Sep 2024 01:37:34 +0900 Subject: [PATCH] blueprint: return nil instead of empty user slice in GetUsers() Callers of GetUser() do not expect to receive an empty slice, which is causing unexpected condition handling. --- pkg/blueprint/customizations.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/blueprint/customizations.go b/pkg/blueprint/customizations.go index 23addb9c9..d361626f5 100644 --- a/pkg/blueprint/customizations.go +++ b/pkg/blueprint/customizations.go @@ -228,7 +228,7 @@ func (c *Customizations) GetTimezoneSettings() (*string, []string) { } func (c *Customizations) GetUsers() []UserCustomization { - if c == nil { + if c == nil || (c.SSHKey == nil && c.User == nil) { return nil }