send empty object in body for requests other than GET (#2589)

aws · Mar 21, 2019 · 917e10c · 917e10c
1 parent 7bff3d5
commit 917e10c
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 3 deletions.
diff --git a/.changes/next-release/bugfix-Paginator-cdfd161d.json b/.changes/next-release/bugfix-Paginator-cdfd161d.json
@@ -0,0 +1,5 @@
+{
+  "type": "bugfix",
+  "category": "Paginator",
+  "description": "still send '{}' in body for requests other than GET"
+}
diff --git a/lib/protocol/rest_json.js b/lib/protocol/rest_json.js
@@ -25,7 +25,7 @@ function populateBody(req) {
     }
   } else {
     var body = builder.build(req.params, input);
-    if (body !== '{}') { //don't send empty object
+    if (body !== '{}' || req.httpRequest.method !== 'GET') { //don't send empty body for GET method
       req.httpRequest.body = body;
     }
     applyContentTypeHeader(req);

diff --git a/test/protocol/rest_json.spec.js b/test/protocol/rest_json.spec.js