websocket url conversion fix (#8)

- adding conversion to the socket url to run in localhost (6024ae64bef7caf5ec656937eab62d9f19410f41)


Co-authored-by: Dmitri Moscoglo <[email protected]>

src/frontend/gradio/app.py

@@ -71,9 +71,14 @@ def get_proxy_url(for_client: bool = False) -> str:
             proxy_url = proxy_url.replace("websocket_proxy", "localhost")
     return proxy_url
 
-def get_proxy_base_url() -> str:
-    """Get HTTP base URL for proxy API calls."""
-    proxy_url = get_proxy_url(for_client=False)
+def get_proxy_base_url(for_client: bool = True) -> str:
+    """Get HTTP base URL for proxy API calls.
+    
+    Args:
+        for_client: If True, returns URL accessible from browser (localhost).
+                   If False, returns internal Docker URL (websocket_proxy).
+    """
+    proxy_url = get_proxy_url(for_client=for_client)
     return proxy_url.replace("ws://", "http://").replace("wss://", "https://").replace("/ws/realtime", "")
 
 def authenticate_voice_screening(email: str, auth_code: str) -> Tuple[str, Optional[str], Optional[str]]:
@@ -81,7 +86,9 @@ def authenticate_voice_screening(email: str, auth_code: str) -> Tuple[str, Optio
     if not email or not auth_code:
         return "❌ Please enter both email and authentication code.", None, None
     try:
-        proxy_base = get_proxy_base_url()
+        # Use for_client=True since Gradio app may run locally and needs to connect to localhost:8000
+        # If running in Docker, the environment variable should be set to use internal URL
+        proxy_base = get_proxy_base_url(for_client=True)
         response = requests.post(
             f"{proxy_base}/auth/verify",
             json={"email": email, "code": auth_code},
@@ -95,6 +102,9 @@ def authenticate_voice_screening(email: str, auth_code: str) -> Tuple[str, Optio
         else:
             error_data = response.json() if response.content else {}
             return f"❌ Authentication failed: {error_data.get('detail', response.text)}", None, None
+    except requests.exceptions.ConnectionError as e:
+        proxy_base = get_proxy_base_url(for_client=True)
+        return f"❌ Error connecting to proxy at {proxy_base}. Make sure the websocket_proxy service is running on port 8000.\n\nIf using Docker, ensure the service is started: `docker compose -f docker/docker-compose.yml up websocket_proxy`", None, None
     except Exception as e:
         return f"❌ Error connecting to proxy: {str(e)}", None, None
 

src/frontend/streamlit/voice_screening_ui/app.py

@@ -46,11 +46,20 @@ def get_proxy_url(for_client=False):
             
     return proxy_url
 
-def get_proxy_base_url():
-    """Get HTTP base URL for proxy API calls (server-side)."""
-    # Use internal URL for server-side requests
-    proxy_url = get_proxy_url(for_client=False)
-    return proxy_url.replace("ws://", "http://").replace("wss://", "https://").replace("/ws/realtime", "")
+def get_proxy_base_url(for_client: bool = False):
+    """Get HTTP base URL for proxy API calls.
+    
+    Args:
+        for_client: If True, returns URL accessible from browser (localhost).
+                   If False, returns internal Docker URL (websocket_proxy).
+    """
+    proxy_url = get_proxy_url(for_client=for_client)
+    # Convert WebSocket URL to HTTP URL
+    base_url = proxy_url.replace("ws://", "http://").replace("wss://", "https://").replace("/ws/realtime", "")
+    # Ensure we have a protocol
+    if not base_url.startswith(("http://", "https://")):
+        base_url = f"http://{base_url}"
+    return base_url
 
 def get_backend_url():
     """Get backend API URL from environment or default."""
@@ -98,9 +107,15 @@ if not st.session_state.session_token:
         if verify_submitted:
             if user_email and auth_code:
                 try:
-                    proxy_base = get_proxy_base_url()
+                    # Use for_client=True to get localhost URL when running outside Docker
+                    # The Streamlit app might be accessed from browser, so use client-accessible URL
+                    proxy_base = get_proxy_base_url(for_client=True)
+                    # Ensure URL has protocol
+                    if not proxy_base.startswith(("http://", "https://")):
+                        proxy_base = f"http://{proxy_base}"
+                    auth_url = f"{proxy_base}/auth/verify"
                     response = requests.post(
-                        f"{proxy_base}/auth/verify",
+                        auth_url,
                         json={"email": user_email, "code": auth_code},
                         timeout=5
                     )
@@ -265,7 +280,11 @@ if st.session_state.is_interview_active:
             # Proxy health check
             if HAS_REQUESTS:
                 try:
-                    proxy_base = get_proxy_base_url()
+                    # Use for_client=True to get localhost URL when running outside Docker
+                    proxy_base = get_proxy_base_url(for_client=True)
+                    # Ensure URL has protocol
+                    if not proxy_base.startswith(("http://", "https://")):
+                        proxy_base = f"http://{proxy_base}"
                     health_url = f"{proxy_base}/health"
                     response = requests.get(health_url, timeout=2)
                     if response.status_code == 200: