Fixes panic during state version upload after some create errors

brandonc · brandonc · commit aaf99193c8f4 · 2023-10-12T09:37:30.000-06:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,9 +1,8 @@
-# UNRELEASED
-<!-- Add CHANGELOG entry to this section for any PR awaiting the next release -->
-<!-- Please also include if this is a Bug Fix, Enhancement, or Feature -->
+# v.1.35.1
 
 ## Bug Fixes
 * Fix incorrect attribute type for `RegistryModule.VCSRepo.Tags` by @hashimoon [#789](https://github.com/hashicorp/go-tfe/pull/789)
+* Fix nil dereference panic within `StateVersions` `upload` after not handling certain state version create errors
 
 # v.1.35.0
 ## Features
diff --git a/state_version.go b/state_version.go
@@ -287,6 +287,7 @@ func (s *stateVersions) Upload(ctx context.Context, workspaceID string, options
 		if strings.Contains(err.Error(), "param is missing or the value is empty: state") {
 			return nil, ErrStateVersionUploadNotSupported
 		}
+		return nil, err
 	}
 
 	g, _ := errgroup.WithContext(ctx)

Original file line number	Diff line number	Diff line change
`@@ -287,6 +287,7 @@ func (s *stateVersions) Upload(ctx context.Context, workspaceID string, options`
`287`	`287`	`if strings.Contains(err.Error(), "param is missing or the value is empty: state") {`
`288`	`288`	`return nil, ErrStateVersionUploadNotSupported`
`289`	`289`	`}`
	`290`	`+ return nil, err`
`290`	`291`	`}`
`291`	`292`
`292`	`293`	`g, _ := errgroup.WithContext(ctx)`