Normalize src path to an absolute path

slug.go

@@ -158,6 +158,12 @@ func (p *Packer) Pack(src string, w io.Writer) (*Meta, error) {
 		ignoreRules = parseIgnoreFile(src)
 	}
 
+	// Ensure the source path provided is absolute
+	src, err = filepath.Abs(src)
+	if err != nil {
+		return nil, fmt.Errorf("failed to read absolute path for source: %w", err)
+	}
+
 	// Walk the tree of files.
 	err = filepath.Walk(src, p.packWalkFn(src, src, src, tarW, meta, ignoreRules))
 	if err != nil {
@@ -264,7 +270,7 @@ func (p *Packer) packWalkFn(root, src, dst string, tarW *tar.Writer, meta *Meta,
 			// If the target is a directory we can recurse into the target
 			// directory by calling the packWalkFn with updated arguments.
 			if resolved.info.IsDir() {
-				return filepath.Walk(resolved.absTarget, p.packWalkFn(root, resolved.target, path, tarW, meta, ignoreRules))
+				return filepath.Walk(resolved.absTarget, p.packWalkFn(root, resolved.absTarget, path, tarW, meta, ignoreRules))
 			}
 
 			// Dereference this symlink by updating the header with the target file

slug_test.go

@@ -55,6 +55,37 @@ func TestPack_rootIsSymlink(t *testing.T) {
 	}
 }
 
+func TestPack_absoluteSrcRelativeSymlinks(t *testing.T) {
+	var path string
+	var err error
+
+	// In instances we run within CI, we want to fetch
+	// the absolute path where our test is located
+	if workDir, ok := os.LookupEnv("GITHUB_WORKSPACE"); ok {
+		path = workDir
+	} else {
+		path, err = os.Getwd()
+		if err != nil {
+			t.Fatalf("could not determine the working dir: %v", err)
+		}
+	}
+
+	// One last check, if this variable is empty we'll error
+	// since we need the absolute path for the source
+	if path == "" {
+		t.Fatal("Home directory could not be determined")
+	}
+
+	path = filepath.Join(path, "testdata/archive-dir-absolute/dev")
+	slug := bytes.NewBuffer(nil)
+	_, err = Pack(path, slug, true)
+	if err != nil {
+		// We simply want to ensure paths can be resolved while
+		// traversing the source directory
+		t.Fatalf("err: %v", err)
+	}
+}
+
 func TestPackWithoutIgnoring(t *testing.T) {
 	slug := bytes.NewBuffer(nil)
 
@@ -1088,8 +1119,8 @@ func assertArchiveFixture(t *testing.T, slug *bytes.Buffer, got *Meta) {
 		}
 
 		if hdr.Name == "example.tf" {
-			if hdr.Typeflag != tar.TypeSymlink {
-				t.Fatalf("expected symlink file 'example.tf'")
+			if hdr.Typeflag != tar.TypeReg {
+				t.Fatalf("expected symlink to be dereferenced 'example.tf'")
 			}
 			externalTargetFound = true
 		}
@@ -1100,9 +1131,9 @@ func assertArchiveFixture(t *testing.T, slug *bytes.Buffer, got *Meta) {
 		t.Fatal("expected to find symlink")
 	}
 
-	// Make sure we saw and handled a nested symlink
+	// Make sure we saw and handled a dereferenced symlink
 	if !externalTargetFound {
-		t.Fatal("expected to find nested symlink")
+		t.Fatal("expected to find dereferenced symlink")
 	}
 
 	// Make sure the .git directory is ignored

testdata/archive-dir-absolute/_common/extra-files/bar.sh

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+echo "bar"

testdata/archive-dir-absolute/_common/extra-files/foo.sh

@@ -0,0 +1,2 @@
+#!/usr/bin/env bash
+echo "foo"

testdata/archive-dir-absolute/_common/locals.tf

@@ -0,0 +1,3 @@
+locals {
+  app = "service-01"
+}

testdata/archive-dir-absolute/_common/output.tf

@@ -0,0 +1,7 @@
+locals {
+    files = fileset("${path.module}/extra-files", "*.sh")
+}
+
+output "scripts" {
+    value = local.files
+}

testdata/archive-dir-absolute/_common/versions.tf

@@ -0,0 +1,3 @@
+terraform {
+  required_version = "~> 1.2"
+}

testdata/archive-dir-absolute/dev/backend.tf

@@ -0,0 +1,10 @@
+terraform {
+  remote "backend" {
+    hostname     = "foobar.terraform.io"
+    organization = "hashicorp"
+
+    workspaces {
+      name = "dev-service-02"
+    }
+  }
+}

testdata/archive-dir-absolute/dev/extra-files

@@ -0,0 +1 @@
+../_common/extra-files

testdata/archive-dir-absolute/dev/locals.tf

@@ -0,0 +1 @@
+../_common/locals.tf

testdata/archive-dir-absolute/dev/output.tf

@@ -0,0 +1 @@
+../_common/output.tf

testdata/archive-dir-absolute/dev/variables.tf

@@ -0,0 +1,4 @@
+locals {
+  env    = "dev"
+  region = "us-east-2"
+}

testdata/archive-dir-absolute/dev/versions.tf

@@ -0,0 +1 @@
+../_common/versions.tf