Support empty env var when it can't be unset

Signed-off-by: David Gageot <[email protected]>

Author: dgageot

build/git.go

@@ -15,8 +15,8 @@ import (
 const DockerfileLabel = "com.docker.image.source.entrypoint"
 
 func addGitProvenance(ctx context.Context, contextPath string, dockerfilePath string) (map[string]string, error) {
-	v, ok := os.LookupEnv("BUILDX_GIT_LABELS")
-	if !ok || contextPath == "" {
+	v := os.Getenv("BUILDX_GIT_LABELS")
+	if (v != "1" && v != "full") || contextPath == "" {
 		return nil, nil
 	}
 	labels := make(map[string]string, 0)

build/git_test.go

@@ -64,6 +64,14 @@ func TestAddGitProvenanceDataWithoutEnv(t *testing.T) {
 	assert.Nilf(t, labels, "No labels expected")
 }
 
+func TestAddGitProvenanceDataWitEmptyEnv(t *testing.T) {
+	defer setupTest(t)(t)
+	os.Setenv("BUILDX_GIT_LABELS", "")
+	labels, err := addGitProvenance(context.Background(), repoDir, filepath.Join(repoDir, "Dockerfile"))
+	assert.Nilf(t, err, "No error expected")
+	assert.Nilf(t, labels, "No labels expected")
+}
+
 func TestAddGitProvenanceDataWithoutLabels(t *testing.T) {
 	defer setupTest(t)(t)
 	os.Setenv("BUILDX_GIT_LABELS", "full")