feat(auth): enhance session handling and response logging in authentication

- Improved session management by ensuring the session is saved before sending the response in the `AuthController`.
- Added detailed logging for session save operations and included the session ID in the response for better debugging.
- Updated session middleware configuration to enhance security and support mobile applications, including adjustments to cookie settings based on the production environment.

worklenz-backend/src/controllers/auth-controller.ts

@@ -259,11 +259,22 @@ export default class AuthController extends WorklenzControllerBase {
         user.build_v = FileConstants.getRelease();
         
         console.log("Sending response...");
-        return res.status(200).send({
+        console.log("Response headers before send:", res.getHeaders());
-          done: true,
+        
-          message: "Login successful",
+        // Ensure session is saved before sending response
-          user,
+        req.session.save((saveErr) => {
-          authenticated: true
+          if (saveErr) {
+            console.log("Session save error:", saveErr);
+          }
+          console.log("Session saved, cookie header:", res.getHeader('set-cookie'));
+          
+          return res.status(200).send({
+            done: true,
+            message: "Login successful",
+            user,
+            authenticated: true,
+            sessionId: req.sessionID // Include for debugging
+          });
         });
       });
     })(req, res, next);

worklenz-backend/src/middlewares/session-middleware.ts

@@ -5,7 +5,7 @@ import { isProduction } from "../shared/utils";
 // eslint-disable-next-line @typescript-eslint/no-var-requires
 const pgSession = require("connect-pg-simple")(session);
 
-export default session({
+const sessionConfig = {
   name: process.env.SESSION_NAME,
   secret: process.env.SESSION_SECRET || "development-secret-key",
   proxy: false,
@@ -18,10 +18,18 @@ export default session({
   }),
   cookie: {
     path: "/",
-    // secure: isProduction(),
+    httpOnly: true,
-    // httpOnly: isProduction(),
+    // For mobile app support, we might need these settings:
-    // sameSite: "none",
+    sameSite: isProduction() ? "none" as const : "lax" as const,
-    // domain: isProduction() ? ".worklenz.com" : undefined,
+    secure: isProduction(), // Required when sameSite is "none"
+    domain: isProduction() ? ".worklenz.com" : undefined,
     maxAge: 30 * 24 * 60 * 60 * 1000 // 30 days
   }
-});
+};
+
+console.log("Session configuration:", {
+  ...sessionConfig,
+  secret: "[REDACTED]"
+});
+
+export default session(sessionConfig);