Do not set a body on a 204, it never makes sense to have it and it of…

…ten causes problems everywhere.

CHANGELOG.md

@@ -3,6 +3,7 @@ This is the changelog for [OpenAPI Factory](readme.md).
 
 ## 5.4 ##
 * Invert the path map. This actually isn't a breaking change, but you are directly using the internals of this library, they are changing.
+* 204 response status codes shall never include the body.
 
 ## 5.3 ##
 * Upgrade to node 14

src/response.js

@@ -1,10 +1,10 @@
 class Response {
   constructor(body, statusCode, headers) {
     let pullBody = body && (body.body || body.statusCode || body.headers);
-    this.body = pullBody ? body.body : body;
     this.statusCode = (pullBody ? body.statusCode : statusCode) || 200;
     let populatedHeaders = (pullBody ? body.headers : headers) || {};
     this.isBase64Encoded = (pullBody ? body.isBase64Encoded : false) || false;
+    this.body = this.statusCode === 204 ? undefined : (pullBody ? body.body : body);
 
     if (!this.body) {
       delete this.body;