Merge branch 'master' into remove-dialog-action-from-lex-event

Author: bmoffatt

.travis.yml

@@ -27,10 +27,10 @@ before_script:
   - LINT_PKGS=$(go list ./... | grep -Ev 'aws-lambda-go/lambda')
 
 script:
-  - diff -u <(echo -n) <(gofmt -d ./)                              # Fail if a .go file hasn't been formatted with gofmt
-  - goverage -v -covermode=atomic -coverprofile=coverage.txt $PKGS # Run all tests with coverage
-  - go vet -v ./...                                                # static analyisis
-  - golint $LINT_PKGS                                              # lint - ignore failures for now
+  - diff -u <(echo -n) <(gofmt -d ./)                                    # Fail if a .go file hasn't been formatted with gofmt
+  - goverage -v -race -covermode=atomic -coverprofile=coverage.txt $PKGS # Run all tests with coverage
+  - go vet -v ./...                                                      # static analyisis
+  - golint $LINT_PKGS                                                    # lint - ignore failures for now
 
 after_success:
   - bash <(curl -s https://codecov.io/bash)

README.md

@@ -54,6 +54,7 @@ Windows developers may have trouble producing a zip file that marks the binary a
 
 Get the tool
 ``` shell
+set GO111MODULE=on
 go.exe get -u github.com/aws/aws-lambda-go/cmd/build-lambda-zip
 ```
 

cmd/build-lambda-zip/main.go

@@ -16,7 +16,7 @@ func main() {
 	app.Name = "build-lambda-zip"
 	app.Usage = "Put an executable and supplemental files into a zip file that works with AWS Lambda."
 	app.Flags = []cli.Flag{
-		cli.StringFlag{
+		&cli.StringFlag{
 			Name:  "output, o",
 			Value: "",
 			Usage: "output file path for the zip. Defaults to the first input file name.",

events/README.md

@@ -14,6 +14,10 @@ This package provides input types for Lambda functions that process AWS events.
 
 [CloudFormation Events](../cfn/README.md)
 
+[CloudWatch Events](README_CloudWatch_Events.md)
+
+[CloudWatch Logs](README_CloudWatch_Logs.md)
+
 [Chime Bot Events](README_Chime_Bots.md)
 
 [Code Commit Events](README_CodeCommit.md)

events/README_CloudWatch_Events.md

@@ -0,0 +1,17 @@
+
+# Sample Function
+
+The following is a Lambda function that receives Amazon CloudWatch event record data as input and writes event detail to Lambda's CloudWatch Logs. Note that by default anything written to Console will be logged as CloudWatch Logs events.
+
+```go
+import (
+	"context"
+	"fmt"
+
+	"github.com/aws/aws-lambda-go/events"
+)
+
+func handler(ctx context.Context, event events.CloudWatchEvent) {
+	fmt.Printf("Detail = %s\n", event.Detail)
+}
+```

events/README_CloudWatch_Logs.md

@@ -0,0 +1,20 @@
+
+# Sample Function
+
+The following is a Lambda function that receives Amazon CloudWatch Logs event record data as input and writes message part to Lambda's CloudWatch Logs. Note that by default anything written to Console will be logged as CloudWatch Logs events.
+
+```go
+import (
+	"context"
+	"fmt"
+
+	"github.com/aws/aws-lambda-go/events"
+)
+
+func handler(ctx context.Context, logsEvent events.CloudwatchLogsEvent) {
+	data, _ := logsEvent.AWSLogs.Parse()
+	for _, logEvent := range data.LogEvents {
+		fmt.Printf("Message = %s\n", logEvent.Message)
+  	}
+}
+```

events/apigw.go

@@ -168,6 +168,7 @@ type APIGatewayCustomAuthorizerPolicy struct {
 	Statement []IAMPolicyStatement
 }
 
+// IAMPolicyStatement represents one statement from IAM policy with action, effect and resource
 type IAMPolicyStatement struct {
 	Action   []string
 	Effect   string

events/attributevalue.go

@@ -75,7 +75,12 @@ func (av DynamoDBAttributeValue) Number() string {
 // of an int64 of the appropriate sign.
 // Method panics if the attribute is not of type Number.
 func (av DynamoDBAttributeValue) Integer() (int64, error) {
-	s, err := strconv.ParseFloat(av.Number(), 64)
+	number := av.Number()
+	value, err := strconv.ParseInt(number, 10, 64)
+	if err == nil {
+		return value, nil
+	}
+	s, err := strconv.ParseFloat(number, 64)
 	return int64(s), err
 }
 

events/attributevalue_test.go

@@ -211,6 +211,7 @@ func TestAccessWithWrongTypePanics(t *testing.T) {
 	}
 
 	for _, testCase := range testCases {
+		testCase := testCase
 		var av DynamoDBAttributeValue
 		err := json.Unmarshal([]byte(testCase.input), &av)
 		assert.Nil(t, err)

events/chime_bot_test.go

@@ -4,10 +4,10 @@ package events
 import (
 	"encoding/json"
 	"testing"
+	"time"
 
 	"github.com/aws/aws-lambda-go/events/test"
 	"github.com/stretchr/testify/assert"
-	"time"
 )
 
 func TestChimeBotEventMarshaling(t *testing.T) {
@@ -116,6 +116,7 @@ func TestChimeBotEventMarshaling(t *testing.T) {
 	}
 
 	for name, test := range tests {
+		test := test
 		t.Run(name, func(t *testing.T) {
 			var testEvent ChimeBotEvent
 			if err := json.Unmarshal([]byte(test.inputJSON), &testEvent); err != nil {