cybercinch/go-semantic-release
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 14 Wiki Activity

ref(internal/releaser/util): check if asset name ist not empty, open file which should be zipped before create zip file

Browse Source
This commit is contained in:
fwiedmann
2019-07-22 18:11:48 +02:00
parent 7d68d5835f
commit 581b62d324
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
internal/releaser/util/util.go
View File

@@ -45,7 +45,9 @@ func GetAccessToken(providerName string) (string, error) {
func PrepareAssets(repository string, assets []config.Asset) ([]*string, error) {
filesToUpload := []*string{}
for _, asset := range assets {
if asset.Compress {
if asset.Name == "" {
return nil, fmt.Errorf("Asset name declaration is empty, please check your configuration file")
} else if asset.Compress {
log.Debugf("Asset %s will now be compressed", asset.Name)
log.Debugf("Repo url %s", repository)
zipNameWithPath, err := zipFile(repository, asset.Name)
@@ -65,6 +67,12 @@ func PrepareAssets(repository string, assets []config.Asset) ([]*string, error)
// ZipFile compress given file in zip format
func zipFile(repository string, file string) (string, error) {
fileToZip, err := os.Open(repository + "/" + file)
if err != nil {
return "", err
}
defer fileToZip.Close()
zipFileName := fmt.Sprintf("%s/%s.zip", strings.TrimSuffix(repository, "/"), file)
zipFile, err := os.Create(zipFileName)
@@ -75,12 +83,6 @@ func zipFile(repository string, file string) (string, error) {
defer zipFile.Close()
fileToZip, err := os.Open(repository + "/" + file)
if err != nil {
return "", err
}
defer fileToZip.Close()
fileToZipInfo, err := fileToZip.Stat()
if err != nil {
return "", err