Merge pull request #3 from fdciabdul/main

fix: align JWT secret default, correct frontend backend URL, validate…

Author: dalpan

backend/routes/webhooks.py

@@ -1,5 +1,6 @@
 import logging
 from typing import Any
+from urllib.parse import urlparse
 
 import httpx
 from fastapi import APIRouter, Depends, HTTPException
@@ -37,9 +38,14 @@ async def create_webhook(data: dict[str, Any], current_user: User = Depends(get_
     if current_user.role not in ("admin", "instructor"):
         raise HTTPException(status_code=403, detail="Insufficient permissions")
 
+    url = data.get("url", "")
+    parsed = urlparse(url)
+    if parsed.scheme not in ("http", "https") or not parsed.netloc:
+        raise HTTPException(status_code=400, detail="Webhook URL must be http(s) with a host")
+
     webhook = WebhookConfig(
         name=data["name"],
-        url=data["url"],
+        url=url,
         events=data.get("events", ["simulation_complete"]),
         secret=data.get("secret"),
         organization_id=current_user.organization_id,

backend/services/auth.py

@@ -9,7 +9,7 @@
 from models.schemas import User
 from services.database import db
 
-JWT_SECRET = os.environ.get("JWT_SECRET", "soceng-lab-secret-key-change-in-production")
+JWT_SECRET = os.environ.get("JWT_SECRET", "change-this-secret-key-in-production")
 JWT_ALGORITHM = "HS256"
 JWT_EXPIRATION_HOURS = 24
 

frontend/.env

@@ -1,2 +1,2 @@
 # REACT_APP_BACKEND_URL
-REACT_APP_BACKEND_URL="http://localhost:8001"
+REACT_APP_BACKEND_URL="http://localhost:9442"