Fixed 500 caused by empty body. (#1964)

Author: limingxinleo

src/Server/Request.php

@@ -480,8 +480,8 @@ protected static function normalizeParsedBody(array $data = [], ?RequestInterfac
 
         try {
             $parser = static::getParser();
-            if ($parser->has($contentType)) {
-                $data = $parser->parse($request->getBody()->getContents(), $contentType);
+            if ($parser->has($contentType) && $content = $request->getBody()->getContents()) {
+                $data = $parser->parse($content, $contentType);
             }
         } catch (\InvalidArgumentException $exception) {
             throw new BadRequestHttpException($exception->getMessage());

tests/ServerRequestTest.php

@@ -93,6 +93,22 @@ public function testXmlNormalizeParsedBodyException()
         $this->assertSame([], RequestStub::normalizeParsedBody($json, $request));
     }
 
+    public function testNormalizeEmptyBody()
+    {
+        $this->getContainer();
+
+        $json = ['name' => 'Hyperf'];
+        $request = Mockery::mock(RequestInterface::class);
+        $request->shouldReceive('getHeaderLine')->with('content-type')->andReturn('application/json; charset=utf-8');
+        $request->shouldReceive('getBody')->andReturn(new SwooleStream(''));
+        $this->assertSame($json, RequestStub::normalizeParsedBody($json, $request));
+
+        $request = Mockery::mock(RequestInterface::class);
+        $request->shouldReceive('getHeaderLine')->with('content-type')->andReturn('application/json; charset=utf-8');
+        $request->shouldReceive('getBody')->andReturn(new SwooleStream(''));
+        $this->assertSame([], RequestStub::normalizeParsedBody([], $request));
+    }
+
     public function testNormalizeParsedBodyInvalidContentType()
     {
         $this->getContainer();