refactor: code reviews

goto · Dec 24, 2024 · ae5b295 · ae5b295
1 parent 0ffbd98
commit ae5b295
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/plugins/providers/oss/provider.go b/plugins/providers/oss/provider.go
@@ -468,26 +468,26 @@ func getPrincipalFromAccountID(accountID, accountType string) (string, error) {
 	if accountType == AccountTypeRAMUser {
 		accountIDParts := strings.Split(accountID, "$")
 		if len(accountIDParts) < 2 {
-			return "", fmt.Errorf("invalid accountID format")
+			return "", fmt.Errorf("invalid accountID format: %q", accountID)
 		}
 
 		subParts := strings.Split(accountIDParts[1], ":")
 		if len(subParts) < 2 {
-			return "", fmt.Errorf("invalid accountID format")
+			return "", fmt.Errorf("invalid accountID format: %q", accountID)
 		}
 
 		return subParts[1], nil
 	} else if accountType == AccountTypeRAMRole {
 
 		accountIDParts := strings.Split(accountID, ":")
 		if len(accountIDParts) < 5 {
-			return "", fmt.Errorf("invalid accountID format")
+			return "", fmt.Errorf("invalid accountID format: %q", accountID)
 		}
 
 		mainAccountID := accountIDParts[3]
 		roleNameParts := strings.Split(accountIDParts[4], "/")
 		if len(roleNameParts) < 2 {
-			return "", fmt.Errorf("invalid accountID format")
+			return "", fmt.Errorf("invalid accountID format: %q: %q", accountID)
 		}
 
 		roleName := roleNameParts[1]