From bc440ce94dd21526e8a0e84c918a26af78eaa313 Mon Sep 17 00:00:00 2001
From: Daniel Kimsey <daniel.kimsey@hashicorp.com>
Date: Wed, 22 Mar 2023 11:02:31 -0500
Subject: [PATCH] Clean up some deprecations and logging behavior

---
 util/lambda_trigger/lambda_trigger.go | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/util/lambda_trigger/lambda_trigger.go b/util/lambda_trigger/lambda_trigger.go
index eb21ad83..ce864899 100644
--- a/util/lambda_trigger/lambda_trigger.go
+++ b/util/lambda_trigger/lambda_trigger.go
@@ -5,7 +5,8 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
-	"io/ioutil"
+	"io"
+	"log"
 	"net/http"
 	"os"
 	"regexp"
@@ -93,7 +94,7 @@ func getBrewVersion(product string, brewType string) (string, error) {
 		// This formula|cask may be new. We'll assume so.
 		return "", errBrewVersionNotFound
 	}
-	body, err := ioutil.ReadAll(resp.Body)
+	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return "", err
 	}
@@ -111,7 +112,7 @@ func triggerGithubWorkflow(event *ReleaseEvent) error {
 	// Create dispatch event https://docs.github.com/en/rest/reference/repos#create-a-repository-dispatch-event
 	workflowEndpoint := "https://api.github.com/repos/hashicorp/homebrew-tap/dispatches"
 	postBody := fmt.Sprintf("{\"event_type\": \"version-updated\", \"client_payload\":{\"name\":\"%s\",\"version\":\"%s\",\"cask\":\"%t\"}}", event.Product, event.Version, event.Cask)
-	fmt.Printf("POSTing to Github: %s\n", postBody)
+	log.Printf("POSTing to Github: %s", postBody)
 
 	httpClient := &http.Client{}
 	req, err := http.NewRequest("POST", workflowEndpoint, bytes.NewBufferString(postBody))
@@ -125,8 +126,8 @@ func triggerGithubWorkflow(event *ReleaseEvent) error {
 		return err
 	}
 	defer resp.Body.Close()
-	body, err := ioutil.ReadAll(resp.Body)
-	fmt.Printf("Github Response: %+v", body)
+	body, err := io.ReadAll(resp.Body)
+	log.Printf("Github Response: %+v", body)
 	return err
 }
 
@@ -134,7 +135,7 @@ func triggerGithubWorkflow(event *ReleaseEvent) error {
 func HandleLambdaEvent(snsEvent events.SNSEvent) error {
 	for _, record := range snsEvent.Records {
 		snsRecord := record.SNS
-		fmt.Printf("[%s %s] Message = %s \n", record.EventSource, snsRecord.Timestamp, snsRecord.Message)
+		log.Printf("[%s %s] Message = %s", record.EventSource, snsRecord.Timestamp, snsRecord.Message)
 		message := record.SNS.Message
 
 		// Parse message to ReleaseEvent type
@@ -149,7 +150,7 @@ func HandleLambdaEvent(snsEvent events.SNSEvent) error {
 			if err != nil || version == nil {
 				return err
 			}
-			fmt.Printf("Latest version is %s\n", *version)
+			log.Printf("Latest version is %s", *version)
 			event.Version = *version
 			oldVersion := ""
 			event.Cask = isCask(event.Product)
@@ -167,9 +168,9 @@ func HandleLambdaEvent(snsEvent events.SNSEvent) error {
 			}
 
 			if oldVersion != "" {
-				fmt.Printf("Current formula/cask version is %s\n", oldVersion)
+				log.Printf("Current formula/cask version is %s", oldVersion)
 			} else {
-				fmt.Printf("No previous version found, assuming new formula/cask\n")
+				log.Printf("No previous version found, assuming new formula/cask")
 			}
 
 			if event.Version == oldVersion {