Fixed issue with DELETE http method

chronoxor · Nov 18, 2020 · bd1cc20 · bd1cc20
1 parent 8812bf5
commit bd1cc20
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 13 deletions.
diff --git a/.appveyor.yml b/.appveyor.yml
@@ -1,5 +1,5 @@
 # Specify version format
-version: "5.0.0.{build}"
+version: "5.0.1.{build}"
 
 # Image to use
 image: Visual Studio 2019

diff --git a/source/NetCoreServer/HttpRequest.cs b/source/NetCoreServer/HttpRequest.cs
@@ -800,8 +800,8 @@ internal bool ReceiveBody(byte[] buffer, int offset, int size)
             // Update body size
             _bodySize += size;
 
-            // GET request has no body
-            if ((Method == "HEAD") || (Method == "GET") || (Method == "OPTIONS") || (Method == "TRACE"))
+            // HEAD/GET/DELETE/OPTIONS/TRACE request might have no body
+            if ((Method == "HEAD") || (Method == "GET") || ((Method == "DELETE") && !_bodyLengthProvided) || (Method == "OPTIONS") || (Method == "TRACE"))
             {
                 _bodyLength = 0;
                 _bodySize = 0;

diff --git a/source/NetCoreServer/NetCoreServer.csproj b/source/NetCoreServer/NetCoreServer.csproj
@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <TargetFramework>net5.0</TargetFramework>
-    <Version>5.0.0.0</Version>
+    <Version>5.0.1.0</Version>
     <Authors>Ivan Shynkarenka</Authors>
     <Copyright>Copyright (c) 2019-2020 Ivan Shynkarenka</Copyright>
     <RepositoryUrl>https://github.com/chronoxor/NetCoreServer</RepositoryUrl>

diff --git a/www/api/openapi.yaml b/www/api/openapi.yaml
@@ -117,15 +117,6 @@ paths:
         schema:
           type: string
           example: 'test'
-      requestBody:
-        required: false
-        content:
-          application/json:
-            type: object
-            schema:
-              type: string
-              nullable: true
-              example: null
       responses:
         200:
           description: Success