4.2.1

.github/dependabot.yml

@@ -9,6 +9,7 @@ updates:
       - anton-yurchenko
     labels:
       - dependencies
+
   - package-ecosystem: npm
     directory: "/"
     schedule:
@@ -18,6 +19,7 @@ updates:
       - anton-yurchenko
     labels:
       - dependencies
+
   - package-ecosystem: "github-actions"
     directory: "/"
     schedule:

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [4.2.1] - 2021-12-15
+### Fixed
+- Error uploading an asset during retry loop
+
 ## [4.2.0] - 2021-11-25
 ### Added
 - Retry assets uploads
@@ -228,7 +232,7 @@ This is a major release as most of the code was refactored and some behavior was
 - Control Release Draft through env.var 'DRAFT_RELEASE'
 - Control Release Pre Release through env.var 'PRE_RELEASE'
 
-[Unreleased]: https://github.com/anton-yurchenko/git-release/compare/v4.2.0...HEAD
+[4.2.1]: https://github.com/anton-yurchenko/git-release/compare/v4.2.0...v4.2.1
 [4.2.0]: https://github.com/anton-yurchenko/git-release/compare/v4.1.2...v4.2.0
 [4.1.2]: https://github.com/anton-yurchenko/git-release/compare/v4.1.1...v4.1.2
 [4.1.1]: https://github.com/anton-yurchenko/git-release/compare/v4.1.0...v4.1.1

main.go

@@ -14,7 +14,7 @@ import (
 )
 
 // Version contains current application version
-const Version string = "4.2.0"
+const Version string = "4.2.1"
 
 func init() {
 	log.SetReportCaller(false)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-release",
-  "version": "4.2.0",
+  "version": "4.2.1",
   "description": "A GitHub Action for creating a GitHub Release with Assets and Changelog whenever a new Tag is pushed to the repository.",
   "main": "wrapper.js",
   "directories": {

release/asset.go

@@ -61,15 +61,14 @@ func (a *Asset) Upload(release *Release, cli RepositoriesClient, id int64, errs
 	defer wg.Done()
 	log.WithField("asset", a.Name).Info("uploading asset")
 
-	file, err := os.Open(a.Path)
-	if err != nil {
-		errs <- err
-		return
-	}
-	defer file.Close()
-
 	maxRetries := 4
 	for i := 1; i <= maxRetries; i++ {
+		file, err := os.Open(a.Path)
+		if err != nil {
+			errs <- err
+			return
+		}
+
 		_, _, err = cli.UploadReleaseAsset(
 			context.Background(),
 			release.Slug.Owner,
@@ -81,6 +80,7 @@ func (a *Asset) Upload(release *Release, cli RepositoriesClient, id int64, errs
 			file,
 		)
 
+		file.Close()
 		if err == nil {
 			errs <- nil
 			break