Merge pull request #438 from dadosfera/fix/proxy-urls

FIX: storage api

deploy/helm-chart/values-stg.yaml

@@ -9,7 +9,7 @@ maestro:
   cookie_secret: "ff7bc13823edb2ae50d248e5780bddc9d4b31c36"
   redis_database: "1"
   platform_api_url: https://xs2hkhq07k.execute-api.us-east-1.amazonaws.com
-  storage_explorer_api_url: "http://storage-explorer-{customer}.data-apps.svc.cluster.local:8000/api"
+  storage_explorer_api_url: "https://storage-explorer-{customer}.stg.dadosfera.ai/api"
 
 hostname: maestro.stg.dadosfera.ai
 

src/modules/auth/auth.controller.ts

@@ -502,7 +502,7 @@ export class AuthController {
     const accessToken = req.cookies['ddf-auth'];
     const refreshToken = req.cookies['ddf-refresh-auth'];
     const userId = req.cookies['ddf-user-id'];
-    const resourceHost = req.headers["x-original-url"] as string || "" ;
+    const resourceHost = req.headers["host"]
 
     const hasUserSession = Boolean(accessToken) && Boolean(userId);
     this.logger.info('Has User Session: ' + hasUserSession);

src/modules/storage-explorer/storage-explorer.config.ts

@@ -6,7 +6,7 @@ export const STORAGE_EXPLORER_CONFIG = {
     }
     // Replace {customer_id} placeholder with actual customer ID
     // For local: http://172.17.0.1:8000/api (no placeholder)
-    // For prod: https://storage-explorer-{customer_id}.dadosfera.ai/api
+    // For prod: https://storage-explorer-{customer_id}.stg.dadosfera.ai/api
     return urlTemplate.replace('{customer}', customerName);
   },
   timeout: parseInt(process.env.STORAGE_EXPLORER_TIMEOUT || '30000', 10),

src/modules/storage-explorer/storage-explorer.controller.ts

@@ -3,13 +3,13 @@ import {
   Get,
   Post,
   Put,
+  Delete,
   Param,
   Body,
   Query,
   Inject,
   UseInterceptors,
   UploadedFiles,
-  Headers,
 } from '@nestjs/common';
 import { ApiTags, ApiOperation, ApiConsumes } from '@nestjs/swagger';
 import { FilesInterceptor } from '@nestjs/platform-express';
@@ -47,13 +47,11 @@ export class StorageExplorerController {
   async validateTableName(
     @Body() body: any,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'POST',
       '/tables/validate-name',
       user,
-      
       body,
     );
   }
@@ -64,13 +62,11 @@ export class StorageExplorerController {
   async createTable(
     @Body() body: any,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'POST',
       '/tables/',
       user,
-      
       body,
     );
   }
@@ -81,13 +77,11 @@ export class StorageExplorerController {
   async listTables(
     @Query('page') page: number,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       '/tables/',
       user,
-      
       undefined,
       { page },
     );
@@ -99,7 +93,6 @@ export class StorageExplorerController {
   async getTable(
     @Param('tableId') tableId: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
@@ -115,13 +108,11 @@ export class StorageExplorerController {
     @Param('tableId') tableId: string,
     @Param('datasetId') datasetId: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'POST',
       `/tables/${tableId}/datasets/${datasetId}`,
       user,
-      
     );
   }
 
@@ -131,13 +122,11 @@ export class StorageExplorerController {
   async getTableDatasets(
     @Param('tableId') tableId: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       `/tables/${tableId}/datasets`,
       user,
-      
     );
   }
 
@@ -147,13 +136,11 @@ export class StorageExplorerController {
   async getTableSchema(
     @Param('tableId') tableId: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       `/tables/${tableId}/schema`,
       user,
-      
     );
   }
 
@@ -169,7 +156,6 @@ export class StorageExplorerController {
       'POST',
       `/tables/${tableId}/validate-compatibility/${datasetId}`,
       user,
-      
     );
   }
 
@@ -184,13 +170,11 @@ export class StorageExplorerController {
     @Param('datasetId') datasetId: string,
     @Query('limit') limit: number,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       `/datasets/${datasetId}/preview`,
       user,
-      
       undefined,
       { limit },
     );
@@ -203,13 +187,11 @@ export class StorageExplorerController {
     @Param('datasetId') datasetId: string,
     @Query('force_refresh') forceRefresh: boolean,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       `/datasets/${datasetId}/schema`,
       user,
-      
       undefined,
       { force_refresh: forceRefresh },
     );
@@ -221,13 +203,11 @@ export class StorageExplorerController {
   async listDatasetsByUpload(
     @Param('uploadId') uploadId: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       `/datasets/upload/${uploadId}`,
       user,
-      
     );
   }
 
@@ -238,13 +218,11 @@ export class StorageExplorerController {
     @Param('datasetId') datasetId: string,
     @Body() body: any,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'PUT',
       `/datasets/${datasetId}/refresh-schema`,
       user,
-      
       body,
     );
   }
@@ -261,13 +239,11 @@ export class StorageExplorerController {
     @Query('limit') limit: number,
     @Query('folder_path') folderPath: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       '/storage/uploads/history',
       user,
-      
       undefined,
       { page, limit, folder_path: folderPath },
     );
@@ -279,13 +255,11 @@ export class StorageExplorerController {
   async browseStorage(
     @Query('path') path: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       '/storage/browse',
       user,
-      
       undefined,
       { path },
     );
@@ -300,7 +274,6 @@ export class StorageExplorerController {
     @UploadedFiles() files: Array<Express.Multer.File>,
     @Body('folder_path') folderPath: string,
     @User() user: RequestUser,
-    
   ) {
     // Create FormData to forward files to storage-explorer API
     const formData = new FormData();
@@ -324,7 +297,6 @@ export class StorageExplorerController {
       'POST',
       '/storage/upload/batch',
       user,
-      
       formData,
     );
   }
@@ -335,13 +307,11 @@ export class StorageExplorerController {
   async createFolder(
     @Body() body: any,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'POST',
       '/storage/folder/create',
       user,
-      
       body,
     );
   }
@@ -352,13 +322,11 @@ export class StorageExplorerController {
   async downloadFile(
     @Query('file_path') filePath: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',
       '/storage/download',
       user,
-      
       undefined,
       { file_path: filePath },
     );
@@ -370,7 +338,6 @@ export class StorageExplorerController {
   async getFileMetadata(
     @Query('file_path') filePath: string,
     @User() user: RequestUser,
-    
   ) {
     return this.storageExplorerService.proxy(
       'GET',

src/modules/storage-explorer/storage-explorer.service.ts

@@ -20,7 +20,7 @@ export class StorageExplorerService {
     path: string,
     user: RequestUser,
     body?: any,
-    query?: Record<string, any>
+    query?: Record<string, any>,
   ): Promise<any> {
     // Validate customer_id is present for multi-tenant isolation
     if (!user.customer_id) {
@@ -113,7 +113,7 @@ export class StorageExplorerService {
     }
 
     // Get customer-specific storage-explorer URL
-    const baseUrl = STORAGE_EXPLORER_CONFIG.getUrl(user.customer_name);
+    const baseUrl = STORAGE_EXPLORER_CONFIG.getUrl(user.customer_id);
     const url = new URL(`${baseUrl}${path}`);
 
     // Add query params