Fail when multiple users with the same name found

kajinamit · kajinamit · commit 61ed30b2e413 · 2023-07-28T14:21:50.000+09:00
... to avoid using the wrong user.
diff --git a/modules/openstack/user.go b/modules/openstack/user.go
@@ -95,6 +95,8 @@ func (o *OpenStack) GetUser(
 
 	if len(allUsers) == 0 {
 		return nil, fmt.Errorf(fmt.Sprintf("%s %s", userName, UserNotFound))
+	} else if len(allUsers) > 1 {
+		return nil, fmt.Errorf(fmt.Sprintf("multiple users named \"%s\" found", userName))
 	}
 
 	return &allUsers[0], nil

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,8 @@ func (o *OpenStack) GetUser(`
`95`	`95`
`96`	`96`	`if len(allUsers) == 0 {`
`97`	`97`	`return nil, fmt.Errorf(fmt.Sprintf("%s %s", userName, UserNotFound))`
	`98`	`+ } else if len(allUsers) > 1 {`
	`99`	`+ return nil, fmt.Errorf(fmt.Sprintf("multiple users named \"%s\" found", userName))`
`98`	`100`	`}`
`99`	`101`
`100`	`102`	`return &allUsers[0], nil`