From 7f6e227ac8fc13d0cda94034187b69a3a0cb115a Mon Sep 17 00:00:00 2001 From: Hugo Osvaldo Barrera Date: Fri, 12 Nov 2021 17:07:37 +0000 Subject: [PATCH] Add `omitempty` to ALBTargetGroupRequest.Body If a lambda returns `Body: nil`, then the ALB returns a 502. --- events/alb.go | 2 +- events/testdata/alb-lambda-target-request-headers-only.json | 3 +-- .../alb-lambda-target-request-multivalue-headers.json | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/events/alb.go b/events/alb.go index e6558ddc..7da6547c 100644 --- a/events/alb.go +++ b/events/alb.go @@ -10,7 +10,7 @@ type ALBTargetGroupRequest struct { MultiValueHeaders map[string][]string `json:"multiValueHeaders,omitempty"` RequestContext ALBTargetGroupRequestContext `json:"requestContext"` IsBase64Encoded bool `json:"isBase64Encoded"` - Body string `json:"body"` + Body string `json:"body,omitempty"` } // ALBTargetGroupRequestContext contains the information to identify the load balancer invoking the lambda diff --git a/events/testdata/alb-lambda-target-request-headers-only.json b/events/testdata/alb-lambda-target-request-headers-only.json index bc25e3f1..7c4bf88e 100644 --- a/events/testdata/alb-lambda-target-request-headers-only.json +++ b/events/testdata/alb-lambda-target-request-headers-only.json @@ -21,6 +21,5 @@ "x-imforwards": "20", "x-myheader": "123" }, - "body": "", "isBase64Encoded": false -} \ No newline at end of file +} diff --git a/events/testdata/alb-lambda-target-request-multivalue-headers.json b/events/testdata/alb-lambda-target-request-multivalue-headers.json index 190b4d2d..f3f3070c 100644 --- a/events/testdata/alb-lambda-target-request-multivalue-headers.json +++ b/events/testdata/alb-lambda-target-request-multivalue-headers.json @@ -43,6 +43,6 @@ "123" ] }, - "body": "", + "body": "Some text", "isBase64Encoded": false -} \ No newline at end of file +}