Don't skip symlinks during local source upload

docs/shp_build_upload.md

@@ -17,8 +17,8 @@ In case a source bundle image is defined, the bundling feature is used, which wi
 source code into a bundle container and upload it to the specified container registry. Instead of
 executing using Git in the source step, it will use the container registry to obtain the source code.
 
-	$ shp buildrun upload <build-name>
-	$ shp buildrun upload <build-name> /path/to/repository
+	$ shp build upload <build-name>
+	$ shp build upload <build-name> /path/to/repository
 
 
 ```

pkg/shp/cmd/build/upload.go

@@ -56,8 +56,8 @@ In case a source bundle image is defined, the bundling feature is used, which wi
 source code into a bundle container and upload it to the specified container registry. Instead of
 executing using Git in the source step, it will use the container registry to obtain the source code.
 
-	$ shp buildrun upload <build-name>
-	$ shp buildrun upload <build-name> /path/to/repository
+	$ shp build upload <build-name>
+	$ shp build upload <build-name> /path/to/repository
 `
 
 	// targetBaseDir directory where data will be uploaded.

pkg/shp/streamer/tar.go

@@ -21,7 +21,7 @@ type Tar struct {
 
 // skipPath inspect each path and makes sure it skips files the tar helper can't handle.
 func (t *Tar) skipPath(fpath string, stat fs.FileInfo) bool {
-	if !stat.Mode().IsRegular() {
+	if !stat.Mode().IsRegular() && !(stat.Mode()&fs.ModeSymlink != 0) {
 		return true
 	}
 	if strings.HasPrefix(fpath, path.Join(t.src, ".git")) {

pkg/shp/streamer/util.go

@@ -28,17 +28,33 @@ func writeFileToTar(tw *tar.Writer, src, fpath string, stat fs.FileInfo) error {
 	}
 
 	header.Name = trimPrefix(src, fpath)
-	if err := tw.WriteHeader(header); err != nil {
-		return err
+
+	// Symlink
+	if stat.Mode()&os.ModeSymlink != 0 {
+		target, err := os.Readlink(fpath)
+		if err != nil {
+			return err
+		}
+
+		header.Linkname = target
 	}
 
-	// #nosec G304 intentionally opening file from variable
-	f, err := os.Open(fpath)
-	if err != nil {
+	if err := tw.WriteHeader(header); err != nil {
 		return err
 	}
-	if _, err := io.Copy(tw, f); err != nil {
-		return err
+
+	// Copy regular file content
+	if stat.Mode().IsRegular() {
+		// #nosec G304 intentionally opening file from variable
+		f, err := os.Open(fpath)
+		if err != nil {
+			return err
+		}
+		if _, err := io.Copy(tw, f); err != nil {
+			return err
+		}
+		return f.Close()
 	}
-	return f.Close()
+
+	return nil
 }