Merge pull request #290 from nkubala/colonoscopy

nkubala · web-flow · commit e3e53da1579b · 2019-01-24T14:49:59.000-08:00
Strip colons from file path before creating cache dir
diff --git a/pkg/util/image_utils.go b/pkg/util/image_utils.go
@@ -161,7 +161,8 @@ func GetImage(imageName string, includeLayers bool, cacheDir string) (Image, err
 	if err != nil {
 		return Image{}, err
 	}
-	path, err := getExtractPathForName(RemoveTag(imageName)+"@"+imageDigest.String(), cacheDir)
+	dirName := strings.Replace(RemoveTag(imageName)+"@"+imageDigest.String(), ":", "", -1)
+	path, err := getExtractPathForName(dirName, cacheDir)
 	if err != nil {
 		return Image{}, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,8 @@ func GetImage(imageName string, includeLayers bool, cacheDir string) (Image, err`
`161`	`161`	`if err != nil {`
`162`	`162`	`return Image{}, err`
`163`	`163`	`}`
`164`		`- path, err := getExtractPathForName(RemoveTag(imageName)+"@"+imageDigest.String(), cacheDir)`
	`164`	`+ dirName := strings.Replace(RemoveTag(imageName)+"@"+imageDigest.String(), ":", "", -1)`
	`165`	`+ path, err := getExtractPathForName(dirName, cacheDir)`
`165`	`166`	`if err != nil {`
`166`	`167`	`return Image{}, err`
`167`	`168`	`}`