default appBaseUrl

Author: roymiloh

src/client.ts

@@ -70,6 +70,9 @@ export function createClient(config: CreateClientConfig): Base44Client {
     headers: optionalHeaders,
   } = config;
 
+  // Normalize appBaseUrl to always be a string (empty if not provided or invalid)
+  const normalizedAppBaseUrl = typeof appBaseUrl === "string" ? appBaseUrl : "";
+
   const socketConfig: RoomsSocketConfig = {
     serverUrl,
     mountPath: "/ws-user-apps/socket.io/",
@@ -135,7 +138,7 @@ export function createClient(config: CreateClientConfig): Base44Client {
     functionsAxiosClient,
     appId,
     {
-      appBaseUrl,
+      appBaseUrl: normalizedAppBaseUrl,
       serverUrl,
     }
   );

src/modules/auth.ts

@@ -49,9 +49,7 @@ export function createAuthModule(
         : window.location.href;
 
       // Build the login URL
-      const loginUrl = `${
-        options.appBaseUrl ?? ""
-      }/login?from_url=${encodeURIComponent(redirectUrl)}`;
+      const loginUrl = `${options.appBaseUrl}/login?from_url=${encodeURIComponent(redirectUrl)}`;
 
       // Redirect to the login page
       window.location.href = loginUrl;

src/modules/auth.types.ts

@@ -98,8 +98,8 @@ export interface ResetPasswordParams {
 export interface AuthModuleOptions {
   /** Server URL for API requests. */
   serverUrl: string;
-  /** Optional base URL for the app (used for login redirects). */
-  appBaseUrl?: string;
+  /** Base URL for the app (used for login redirects). */
+  appBaseUrl: string;
 }
 
 /**

tests/unit/client.test.js

@@ -64,7 +64,7 @@ describe('Client Creation', () => {
       serviceToken: 'service-token-123',
       requiresAuth: true,
     });
-    
+
     expect(client).toBeDefined();
     expect(client.entities).toBeDefined();
     expect(client.integrations).toBeDefined();
@@ -78,6 +78,59 @@ describe('Client Creation', () => {
 
 });
 
+describe('appBaseUrl Normalization', () => {
+  test('should use appBaseUrl when provided as a string', () => {
+    const customAppBaseUrl = 'https://custom-app.example.com';
+    const client = createClient({
+      appId: 'test-app-id',
+      appBaseUrl: customAppBaseUrl,
+    });
+
+    // Mock window.location
+    const originalWindow = global.window;
+    const mockLocation = { href: '', origin: 'https://current-app.com' };
+    global.window = {
+      location: mockLocation
+    };
+
+    const nextUrl = 'https://example.com/dashboard';
+    client.auth.redirectToLogin(nextUrl);
+
+    // Verify the redirect URL uses the custom appBaseUrl
+    expect(mockLocation.href).toBe(
+      `${customAppBaseUrl}/login?from_url=${encodeURIComponent(nextUrl)}`
+    );
+
+    // Restore window
+    global.window = originalWindow;
+  });
+
+  test('should normalize appBaseUrl to empty string when not provided', () => {
+    const client = createClient({
+      appId: 'test-app-id',
+      // appBaseUrl not provided
+    });
+
+    // Mock window.location
+    const originalWindow = global.window;
+    const mockLocation = { href: '', origin: 'https://current-app.com' };
+    global.window = {
+      location: mockLocation
+    };
+
+    const nextUrl = 'https://example.com/dashboard';
+    client.auth.redirectToLogin(nextUrl);
+
+    // Verify the redirect URL uses empty string (relative path)
+    expect(mockLocation.href).toBe(
+      `/login?from_url=${encodeURIComponent(nextUrl)}`
+    );
+
+    // Restore window
+    global.window = originalWindow;
+  });
+});
+
 describe('createClientFromRequest', () => {
   test('should create client from request with all headers', () => {
     const mockRequest = {