Replace deprecated GHA ::set-output syntax

Fixes #867
paketo-buildpacks · Jan 24, 2023 · d91eefc · d91eefc
1 parent 55ebdb5
commit d91eefc
Show file tree

Hide file tree

Showing 48 changed files with 95 additions and 101 deletions.
diff --git a/actions/action.go b/actions/action.go
@@ -17,8 +17,11 @@
 package actions
 
 import (
+	"crypto/rand"
+	"encoding/hex"
+	"errors"
 	"fmt"
-	"io"
+	"log"
 	"os"
 	"regexp"
 	"strings"
@@ -62,8 +65,27 @@ func NewOutputs(uri string, latestVersion *semver.Version, additionalOutputs Out
 	return outputs, nil
 }
 
-func (o Outputs) Write(writer io.Writer) {
+func (o Outputs) Write() {
+	outputFileName, ok := os.LookupEnv("GITHUB_OUTPUT")
+	if !ok {
+		panic(errors.New("GITHUB_OUTPUT is not set, see https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter"))
+	}
+	writer, err := os.OpenFile(outputFileName, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0666)
+	if err != nil {
+		panic(err)
+	}
+	defer writer.Close()
+	delimiter := generateDelimiter()
 	for k, v := range o {
-		_, _ = fmt.Fprintf(writer, "::set-output name=%s::%s\n", k, v)
+		_, _ = fmt.Fprintf(writer, "%s<<%s\n%s\n%s\n", k, delimiter, v, delimiter) // see https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#multiline-strings
+	}
+}
+
+func generateDelimiter() string {
+	data := make([]byte, 16) // roughly the same entropy as uuid v4 used in https://github.com/actions/toolkit/blob/b36e70495fbee083eb20f600eafa9091d832577d/packages/core/src/file-command.ts#L28
+	_, err := rand.Read(data)
+	if err != nil {
+		log.Fatal("could not generate random delimiter", err)
 	}
+	return hex.EncodeToString(data)
 }
diff --git a/actions/adoptium-dependency/main.go b/actions/adoptium-dependency/main.go
@@ -21,7 +21,6 @@ import (
 	"fmt"
 	"net/http"
 	"net/url"
-	"os"
 	"strings"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
@@ -93,7 +92,7 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 type Asset struct {

diff --git a/actions/alibaba-dragonwell-dependency/main.go b/actions/alibaba-dragonwell-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"context"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 	"strings"
 
@@ -107,5 +106,5 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
diff --git a/actions/amazon-corretto-dependency/main.go b/actions/amazon-corretto-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"context"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 
 	"github.com/google/go-github/v43/github"
@@ -117,5 +116,5 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
diff --git a/actions/appdynamics-dependency/main.go b/actions/appdynamics-dependency/main.go
@@ -21,7 +21,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
@@ -64,7 +63,7 @@ func main() {
 	if o, err := versions.GetLatest(inputs, addToken); err != nil {
 		panic(fmt.Errorf("unable to get latest\n%w", err))
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
 

diff --git a/actions/aqua-security-dependency/main.go b/actions/aqua-security-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"bufio"
 	"fmt"
 	"net/http"
-	"os"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
 )
@@ -64,6 +63,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs, actions.WithBasicAuth(u, p)); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/aternity-dependency/main.go b/actions/aternity-dependency/main.go
@@ -18,7 +18,6 @@ package main
 
 import (
 	"fmt"
-	"os"
 	"regexp"
 
 	"github.com/aws/aws-sdk-go/aws"
@@ -53,6 +52,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/azul-zulu-dependency/main.go b/actions/azul-zulu-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"os"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
 )
@@ -85,7 +84,7 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 type Bundle struct {

diff --git a/actions/bellsoft-liberica-dependency/main.go b/actions/bellsoft-liberica-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
@@ -124,7 +123,7 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 type Release struct {

diff --git a/actions/ca-apm-dependency/main.go b/actions/ca-apm-dependency/main.go
@@ -18,7 +18,6 @@ package main
 
 import (
 	"fmt"
-	"os"
 	"regexp"
 
 	"github.com/gocolly/colly"
@@ -70,6 +69,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/cf-java-index-dependency/main.go b/actions/cf-java-index-dependency/main.go
@@ -19,7 +19,6 @@ package main
 import (
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
@@ -61,6 +60,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/clojure-tools-dependency/main.go b/actions/clojure-tools-dependency/main.go
@@ -22,7 +22,6 @@ import (
 	"io"
 	"log"
 	"net/http"
-	"os"
 	"strings"
 
 	"github.com/google/go-github/v43/github"
@@ -105,7 +104,7 @@ func main() {
 		if o, err := versions.GetLatest(inputs); err != nil {
 			panic(err)
 		} else {
-			o.Write(os.Stdout)
+			o.Write()
 		}
 	}
 }
diff --git a/actions/compute-artifact-description/main.go b/actions/compute-artifact-description/main.go
@@ -93,7 +93,9 @@ func main() {
 	}
 	sort.Strings(deps)
 
-	fmt.Printf("::set-output name=artifact-reference-description::%s", strings.Join(deps, "%0A")) // linefeed
+	actions.Outputs{
+		"artifact-reference-description": strings.Join(deps, "\n"),
+	}.Write()
 }
 
 func parseMappers(inputs actions.Inputs) []string {

diff --git a/actions/foojay-dependency/main.go b/actions/foojay-dependency/main.go
@@ -21,7 +21,6 @@ import (
 	"fmt"
 	"net/http"
 	"net/url"
-	"os"
 	"regexp"
 	"strconv"
 
@@ -94,7 +93,7 @@ func main() {
 		outputs["cpe"] = fmt.Sprintf("update%d", latestVersion.Patch())
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 func LoadPackages(d string, t string, v int) actions.Versions {

diff --git a/actions/gcs-dependency/main.go b/actions/gcs-dependency/main.go
@@ -19,7 +19,6 @@ package main
 import (
 	"context"
 	"fmt"
-	"os"
 	"regexp"
 
 	"cloud.google.com/go/storage"
@@ -72,6 +71,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/github-release-dependency/main.go b/actions/github-release-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"context"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 	"sort"
 	"strconv"
@@ -131,6 +130,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/google-stackdriver-profiler-dependency/main.go b/actions/google-stackdriver-profiler-dependency/main.go
@@ -19,7 +19,6 @@ package main
 import (
 	"context"
 	"fmt"
-	"os"
 	"regexp"
 
 	"cloud.google.com/go/storage"
@@ -58,6 +57,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/graalvm-dependency/main.go b/actions/graalvm-dependency/main.go
@@ -24,7 +24,6 @@ import (
 	"io"
 	"io/ioutil"
 	"net/http"
-	"os"
 	"regexp"
 	"strings"
 
@@ -132,7 +131,7 @@ func main() {
 		outputs["purl"] = matches[1]
 	}
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 func GetVersion(uri string) string {

diff --git a/actions/gradle-dependency/main.go b/actions/gradle-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"os"
 
 	"github.com/paketo-buildpacks/pipeline-builder/actions"
 )
@@ -53,7 +52,7 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
 

diff --git a/actions/ibm-semeru-dependency/main.go b/actions/ibm-semeru-dependency/main.go
@@ -21,7 +21,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"os"
 	"regexp"
 	"strings"
 
@@ -132,7 +131,7 @@ func main() {
 	}
 	outputs["cpe"] = matches[1]
 
-	outputs.Write(os.Stdout)
+	outputs.Write()
 }
 
 func getReleaseJSON(jsonURI string) ReleaseJSON {

diff --git a/actions/jprofiler-dependency/main.go b/actions/jprofiler-dependency/main.go
@@ -18,7 +18,6 @@ package main
 
 import (
 	"fmt"
-	"os"
 	"regexp"
 
 	"github.com/gocolly/colly"
@@ -49,6 +48,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/jrebel-dependency/main.go b/actions/jrebel-dependency/main.go
@@ -18,7 +18,6 @@ package main
 
 import (
 	"fmt"
-	"os"
 	"regexp"
 
 	"github.com/gocolly/colly"
@@ -49,6 +48,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }
diff --git a/actions/leiningen-dependency/main.go b/actions/leiningen-dependency/main.go
@@ -20,7 +20,6 @@ import (
 	"context"
 	"fmt"
 	"net/http"
-	"os"
 
 	"github.com/google/go-github/v43/github"
 	"golang.org/x/oauth2"
@@ -59,6 +58,6 @@ func main() {
 	if o, err := versions.GetLatest(inputs); err != nil {
 		panic(err)
 	} else {
-		o.Write(os.Stdout)
+		o.Write()
 	}
 }