Update app.py

app.py

@@ -25,16 +25,16 @@ app = FastAPI()
 # CORS Configuration (restrict in production)
 app.add_middleware(
     CORSMiddleware,
-    allow_origins=["http://localhost:7860", "https://your-production-domain.com"],  # Update for production
+    allow_origins=["http://localhost:7860", "https://your-production-domain.com"],
     allow_credentials=True,
     allow_methods=["*"],
     allow_headers=["*"],
 )
 
-# API Router (assuming this is defined elsewhere)
+# API Router (assuming minimal or no conflicting endpoints)
 api_router = APIRouter()
 
-# Constants (load from environment variables for security)
+# Constants (load from environment variables)
 BACKEND_URL = os.getenv("BACKEND_URL", "https://rocketfarmstudios-cps-api.hf.space")
 ADMIN_EMAIL = os.getenv("ADMIN_EMAIL", "yakdhanali97@gmail.com")
 ADMIN_PASSWORD = os.getenv("ADMIN_PASSWORD", "123456")
@@ -42,12 +42,11 @@ MAX_TOKEN_RETRIES = 3
 TOKEN_RETRY_DELAY = 2  # seconds
 TOKEN_EXPIRY = 3600  # 1 hour default expiry
 
-# Pydantic model for login payload
+# Pydantic models
 class LoginPayload(BaseModel):
     username: str
     password: str
 
-# Pydantic model for doctor creation payload
 class DoctorPayload(BaseModel):
     full_name: str
     email: str
@@ -66,11 +65,13 @@ class TokenManager:
         try:
             async with aiohttp.ClientSession() as session:
                 payload = LoginPayload(username=ADMIN_EMAIL, password=ADMIN_PASSWORD)
+                logger.debug(f"Sending login request to {BACKEND_URL}/auth/login with payload: {payload.dict()}")
                 async with session.post(
                     f"{BACKEND_URL}/auth/login",
                     json=payload.dict(),
                     timeout=10
                 ) as response:
+                    logger.debug(f"Login response status: {response.status}")
                     if response.status == 200:
                         data = await response.json()
                         token = data.get("access_token")
@@ -120,7 +121,7 @@ def root():
 @app.post("/login")
 async def redirect_login(request: Request):
     logger.info("Redirecting /login to /auth/login")
-    return RedirectResponse(url="/auth/login", status_code=307)
+    return RedirectResponse(url="/admin-auth", status_code=307)  # Redirect to Gradio UI
 
 def authenticate_admin(email: str = None, password: str = None):
     if email != ADMIN_EMAIL or password != ADMIN_PASSWORD:
@@ -134,6 +135,7 @@ async def async_create_doctor(full_name: str, email: str, license_number: str, s
     try:
         # Validate inputs
         if not all([full_name, email, license_number, specialty, password]):
+            logger.error("Doctor creation failed: All fields are required")
             raise HTTPException(status_code=422, detail="All fields are required")
 
         token = await token_manager.get_token()
@@ -150,6 +152,7 @@ async def async_create_doctor(full_name: str, email: str, license_number: str, s
             "Content-Type": "application/json"
         }
         
+        logger.debug(f"Sending doctor creation request to {BACKEND_URL}/auth/admin/doctors with payload: {payload.dict()}")
         async with aiohttp.ClientSession() as session:
             async with session.post(
                 f"{BACKEND_URL}/auth/admin/doctors",
@@ -157,9 +160,10 @@ async def async_create_doctor(full_name: str, email: str, license_number: str, s
                 headers=headers,
                 timeout=10
             ) as response:
+                logger.debug(f"Doctor creation response status: {response.status}")
                 if response.status == 201:
                     return "✅ Doctor created successfully!"
-                elif response.status == 401:  # Token might be expired
+                elif response.status == 401:
                     logger.warning("Token expired, attempting refresh...")
                     token = await token_manager.refresh_token()
                     headers["Authorization"] = f"Bearer {token}"
@@ -169,6 +173,7 @@ async def async_create_doctor(full_name: str, email: str, license_number: str, s
                         headers=headers,
                         timeout=10
                     ) as retry_response:
+                        logger.debug(f"Retry doctor creation response status: {retry_response.status}")
                         if retry_response.status == 201:
                             return "✅ Doctor created successfully!"
                         error_detail = await retry_response.text()
@@ -220,14 +225,15 @@ with admin_ui:
     gr.Markdown("# Doctor Account Creator")
     
     with gr.Column():
-        full_name = gr.Textbox(label="Full Name")
-        email = gr.Textbox(label="Email")
-        matricule = gr.Textbox(label="License Number")
+        full_name = gr.Textbox(label="Full Name", placeholder="e.g., Dr. John Doe")
+        email = gr.Textbox(label="Email", placeholder="e.g., john.doe@example.com")
+        matricule = gr.Textbox(label="License Number", placeholder="e.g., 12345")
         specialty = gr.Dropdown(
             label="Specialty",
-            choices=["General Practice", "Cardiology", "Neurology", "Pediatrics"]
+            choices=["General Practice", "Cardiology", "Neurology", "Pediatrics"],
+            value="General Practice"
         )
-        password = gr.Textbox(label="Password", type="password")
+        password = gr.Textbox(label="Password", type="password", placeholder="Enter a secure password")
         submit_btn = gr.Button("Create Account")
         output = gr.Textbox(label="Status", interactive=False)
         
@@ -259,7 +265,6 @@ async def gradio_queue_data(session_hash: str):
 
 @app.on_event("startup")
 async def startup_event():
-    """Initialize token but don't fail startup"""
     try:
         await token_manager.get_token()
         logger.info("Initial token fetch successful")