Beejones/service request must show body in debug

src/utils/ServiceRequest.ts

@@ -95,18 +95,16 @@ export class ServiceRequest<T> {
       }
     }
 
-    Logger.debug(`Request:`, this.logContext, JSON.stringify(requestWithoutAuth, null, 2));
-    this.query = queryParams(request, this.logContext);
-
     try {
       this.body = request.body.json();
     } catch (exception) {
-      this.error = {
-        errorMessage: `${this.logContext.getBaseScope()}: No valid JSON request for ${this.logContext.getFormattedScopeString()}`,
-      };
-      this.success = false;
-      return;
+      Logger.info("No JSON body found", this.logContext);
     }
+
+    requestWithoutAuth.body = this.body;
+    Logger.debug(`Request:`, this.logContext, JSON.stringify(requestWithoutAuth, null, 2));
+    this.query = queryParams(request, this.logContext);
+
     this.success = true;
   }
 

test/unit-test/utils/ServiceRequest.test.ts

@@ -74,15 +74,13 @@ describe("Test ServiceRequest properties", () => {
         }));
         settingsPolicyMap.set("settings_policy", settingsPolicyValue);
 
-
         // Mock Logger.debug
         const debugSpy = jest.spyOn(console, "log").mockImplementation(() => true);
 
-
-
         // Act
         const serviceRequest =
             new ServiceRequest<void>(logContext, <any>request);
+        // Dump all messages received by debugSpy
         console.dir(debugSpy.mock.calls, { depth: null });
 
         // Assert
@@ -128,7 +126,9 @@ describe("Test ServiceRequest properties", () => {
             '  "route": "/app/key",\n' +
             '  "url": "/app/key",\n' +
             '  "params": {},\n' +
-            '  "body": {}\n' +
+            '  "body": {\n' +
+            '    "key": "value"\n' +
+            '  }\n' +
             '}';
         expect(debugSpy).toHaveBeenCalledWith(expectedLogMessageWithoutBearer);