Fix case with the string "null" being sent back when using streaming

Author: thekid

ChangeLog.md

@@ -3,6 +3,11 @@ AWS Lambda Webservices change log
 
 ## ?.?.? / ????-??-??
 
+## 1.1.4 / 2023-04-03
+
+* Fixed case with the string "null" being sent back when using streaming
+  and never writing to the response stream by removing response `body`
+  (@thekid)
 * Merged PR #9: Migrate to new testing library - @thekid
 
 ## 1.1.3 / 2022-10-14

src/main/php/com/amazon/aws/lambda/ResponseDocument.class.php

@@ -6,6 +6,7 @@
  * Response document
  *
  * @test  com.amazon.aws.lambda.unittest.ResponseDocumentTest
+ * @see   https://docs.aws.amazon.com/apigateway/latest/developerguide/http-api-develop-integrations-lambda.html
  */
 class ResponseDocument extends Output {
   public $document;
@@ -97,7 +98,10 @@ public function write($bytes) {
 
   /** @return void */
   public function finish() {
-    if (null === $this->document['body']) return;
+    if (null === $this->document['body']) {
+      unset($this->document['body']);
+      return;
+    }
 
     // Report unencoded length in headers
     $this->document['headers']['Content-Length']= (string)strlen($this->document['body']);

src/test/php/com/amazon/aws/lambda/unittest/HttpApiTest.class.php

@@ -127,7 +127,6 @@ public function routes($env) {
         'statusDescription' => 'Found',
         'isBase64Encoded'   => false,
         'headers'           => ['Location' => 'https://example.com', 'Content-Length' => 0],
-        'body'              => null,
       ],
       $this->invoke($fixture->target(), 'GET')
     );

src/test/php/com/amazon/aws/lambda/unittest/ResponseDocumentTest.class.php

@@ -23,7 +23,6 @@ public function no_content() {
         'statusDescription' => 'No Content',
         'isBase64Encoded'   => false,
         'headers'           => [],
-        'body'              => null,
       ],
       $out->document
     );