Merge remote-tracking branch 'origin/primary' into primary

# Conflicts:
#	go.mod
#	go.sum

Author: mousedownmike

README.md

@@ -22,12 +22,12 @@ zip signed.zip signed
 These are the basic steps to create the function using the console.  If you're here, you probably already know how to do this, there's nothing special about this project.
 
 1. Choose **Create Function**
-1. Select **Author from scratch**
-  * Function name: `bucketlist`
-  * Runtime `Go 1.x`
-1. Select **Create Function**
-1. Select your `signed.zip` file after selecting the **Upload** button.
-1. In the **Handler** field enter `signed`
-1. Select the **Save** button in the top right.
+2. Select **Author from scratch**
+   * Function name: `bucketlist`
+   * Runtime `Go 1.x`
+3. Select **Create Function**
+4. Select your `signed.zip` file after selecting the **Upload** button.
+5. In the **Handler** field enter `signed`
+6. Select the **Save** button in the top right.
 
 The function doesn't expect any input so you can configure a simple test event (e.g. `{}`) and run the function by clicking the **Test** button.  The function will succeed but the output will show that access was denied.  You need to add the `s3:ListAllMyBuckets` policy action to your newly created role.  Run the test again and you'll see the bucket list. 

cmd/signed/main.go

@@ -5,7 +5,7 @@ import (
 	"github.com/aws/aws-lambda-go/lambda"
 	"github.com/aws/aws-sdk-go/aws/session"
 	v4 "github.com/aws/aws-sdk-go/aws/signer/v4"
-	"io/ioutil"
+	"io"
 	"log"
 	"net/http"
 	"os"
@@ -30,7 +30,7 @@ func main() {
 		endpoint: S3Endpoint,
 		region:   S3Region,
 		signer:   signer,
-		client:    &http.Client{},
+		client:   &http.Client{},
 	}
 
 	// If this environment variable is set, assume we're running in a Lambda
@@ -57,8 +57,8 @@ type BucketLister struct {
 	client   *http.Client
 }
 
-/**
-Log all the buckets available to the assumed role.
+/*
+List buckets available to the assumed role.
 */
 func (b *BucketLister) List() error {
 	url := fmt.Sprintf("https://%s/", b.endpoint)
@@ -76,8 +76,10 @@ func (b *BucketLister) List() error {
 	if err != nil {
 		return fmt.Errorf("error making request: %w", err)
 	}
-	defer resp.Body.Close()
-	body, err := ioutil.ReadAll(resp.Body)
+	defer func(Body io.ReadCloser) {
+		_ = Body.Close()
+	}(resp.Body)
+	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return fmt.Errorf("error reading resposne: %w", err)
 	}