Merge pull request #611 from versity/ben/iam_s3_nil_map

fix: init auth config accounts map
versity · Jun 10, 2024 · f110649 · f110649
2 parents f6755cb + d5ecb97
commit f110649
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/auth/iam_internal.go b/auth/iam_internal.go
@@ -188,6 +188,10 @@ func parseIAM(b []byte) (iAMConfig, error) {
 		return iAMConfig{}, fmt.Errorf("failed to parse the config file: %w", err)
 	}
 
+	if conf.AccessAccounts == nil {
+		conf.AccessAccounts = make(map[string]Account)
+	}
+
 	return conf, nil
 }
 

diff --git a/auth/iam_s3_object.go b/auth/iam_s3_object.go
@@ -204,12 +204,12 @@ func (s *IAMServiceS3) getAccounts() (iAMConfig, error) {
 		// init empty accounts stuct and return that
 		var nsk *types.NoSuchKey
 		if errors.As(err, &nsk) {
-			return iAMConfig{}, nil
+			return iAMConfig{AccessAccounts: map[string]Account{}}, nil
 		}
 		var apiErr smithy.APIError
 		if errors.As(err, &apiErr) {
 			if apiErr.ErrorCode() == "NotFound" {
-				return iAMConfig{}, nil
+				return iAMConfig{AccessAccounts: map[string]Account{}}, nil
 			}
 		}