Merge pull request #25 from nginxinc/areste-windowstarball

mrajagopal · web-flow · commit edc45291e4b5 · 2024-07-16T10:30:01.000+12:00
Use filepath tools for Windows compatibility
diff --git a/pkg/data_collector/data_collector.go b/pkg/data_collector/data_collector.go
@@ -162,7 +162,7 @@ func (c *DataCollector) WrapUp(product string) (string, error) {
 		if err != nil {
 			return err
 		}
-		header.Name = tarballRootDirName + "/" + relativePath
+		header.Name = filepath.ToSlash(filepath.Join(tarballRootDirName, relativePath))
 
 		if err = tw.WriteHeader(header); err != nil {
 			return err

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ func (c *DataCollector) WrapUp(product string) (string, error) {`
`162`	`162`	`if err != nil {`
`163`	`163`	`return err`
`164`	`164`	`}`
`165`		`- header.Name = tarballRootDirName + "/" + relativePath`
	`165`	`+ header.Name = filepath.ToSlash(filepath.Join(tarballRootDirName, relativePath))`
`166`	`166`
`167`	`167`	`if err = tw.WriteHeader(header); err != nil {`
`168`	`168`	`return err`