kunthawat/ALwrity
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: require REACT_APP_API_URL in production, throw clear error if missing

Browse Source
This commit is contained in:
ajaysi
2026-04-04 07:08:34 +05:30
parent a3e4f5231a
commit c5e2fc3514
7 changed files with 67 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
frontend/src/api/client.ts
View File

@@ -64,13 +64,21 @@ export const getAuthTokenGetter = (): (() => Promise<string | null>) | null => {
// Get API URL from environment variables
export const getApiUrl = () => {
if (process.env.NODE_ENV === 'production') {
// In production, use the environment variable or fallback
return process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL;
const apiUrl = process.env.REACT_APP_API_URL;
const isProduction = process.env.NODE_ENV === 'production';
// In production, require REACT_APP_API_URL to be set
if (isProduction && !apiUrl) {
console.error('[apiClient] ❌ REACT_APP_API_URL is not set for production! Please configure in Vercel environment variables.');
throw new Error('REACT_APP_API_URL environment variable is required for production. Please set it in your Vercel project settings.');
}
// In development, prefer the local backend to avoid CORS/proxy header stripping.
// If an ngrok URL is set in env but we're on localhost, override to localhost:8000.
const envUrl = process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL;
if (isProduction) {
return apiUrl;
}
// In development, use localhost by default
const envUrl = process.env.REACT_APP_API_URL;
const isLocalhost = typeof window !== 'undefined' && window.location.hostname === 'localhost';
const isNgrok = envUrl && envUrl.includes('ngrok');
if (isLocalhost) {

10
frontend/src/hooks/useCollections.ts
View File

@@ -26,7 +26,15 @@ export interface CollectionUpdateRequest {
cover_asset_id?: number;
}
const API_BASE_URL = process.env.REACT_APP_API_URL || process.env.REACT_APP_API_BASE_URL || 'http://localhost:8000';
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
const API_BASE_URL = getApiBaseUrl();
export const useCollections = () => {
const { getToken } = useAuth();

10
frontend/src/hooks/useContentAssets.ts
View File

@@ -49,7 +49,15 @@ export interface AssetListResponse {
offset: number;
}
const API_BASE_URL = process.env.REACT_APP_API_URL || process.env.REACT_APP_API_BASE_URL || 'http://localhost:8000';
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
const API_BASE_URL = getApiBaseUrl();
export const useContentAssets = (filters: AssetFilters = {}) => {
const { getToken } = useAuth();

11
frontend/src/hooks/useRealTimeData.ts
View File

@@ -50,8 +50,15 @@ export const useRealTimeData = (options: RealTimeDataOptions) => {
try {
// Build WebSocket URL from environment variables
// Consistent with API URL pattern - no hardcoded localhost
const apiUrl = process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL || '';
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
const apiUrl = getApiBaseUrl();
// In development, use proxy (empty string means use same origin)
// In production, derive WebSocket URL from API URL

10
frontend/src/services/hallucinationDetectorService.ts
View File

@@ -77,8 +77,14 @@ class HallucinationDetectorService {
private baseUrl: string;
constructor() {
// Consistent API URL pattern - no hardcoded localhost fallback
this.baseUrl = process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL || '';
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
this.baseUrl = getApiBaseUrl();
}
/**

12
frontend/src/services/seoApiService.ts
View File

@@ -12,8 +12,16 @@ import {
CopilotSuggestion
} from '../types/seoCopilotTypes';
// Consistent API URL pattern - use same env vars as other services
const API_BASE_URL = process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL || '';
// API URL - require REACT_APP_API_URL in production
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
const API_BASE_URL = getApiBaseUrl();
class SEOApiService {
private baseUrl: string;

10
frontend/src/services/writingAssistantService.ts
View File

@@ -21,8 +21,14 @@ export interface WASuggestResponse {
class WritingAssistantService {
private baseUrl: string;
constructor() {
// Consistent API URL pattern - no hardcoded localhost fallback
this.baseUrl = process.env.REACT_APP_API_URL || process.env.REACT_APP_BACKEND_URL || '';
const getApiBaseUrl = () => {
const url = process.env.REACT_APP_API_URL;
if (process.env.NODE_ENV === 'production' && !url) {
throw new Error('REACT_APP_API_URL environment variable is required for production');
}
return url || 'http://localhost:8000';
};
this.baseUrl = getApiBaseUrl();
}
async suggest(text: string): Promise<WASuggestion[]> {