fix: update default status code handling in render response to use 200 when statusCode is null or invalid

Author: LabEG

src/controllers/render-controller/render.controller.spec.ts

@@ -161,7 +161,7 @@ describe("RenderController", () => {
             ).rejects.toThrow("Navigation timeout");
         });
 
-        it("should use default status code 500 when statusCode is null", async () => {
+        it("should use default status code 200 when statusCode is null", async () => {
             const mockRequest: MockRequest = {
                 url: "/render/https://example.com",
                 headers: { "user-agent": "test-agent" },
@@ -179,7 +179,7 @@ describe("RenderController", () => {
                 mockResponse as Response,
             );
 
-            expect(mockResponse.status).toHaveBeenCalledWith(500);
+            expect(mockResponse.status).toHaveBeenCalledWith(200);
         });
 
         it("should use default status code 200 when statusCode is invalid", async () => {

src/controllers/render-controller/render.controller.ts

@@ -30,8 +30,9 @@ export class RenderController {
 
         const result = await this.prerenderService.render(url, reguest.headers);
 
-        response
-            .status(Number.parseInt(result.statusCode ?? "200"))
-            .send(result.pageContent);
+        const statusCode = Number.parseInt(result.statusCode ?? "200");
+        const validStatusCode = Number.isNaN(statusCode) ? 200 : statusCode;
+
+        response.status(validStatusCode).send(result.pageContent);
     }
 }