fixup: rework url_list handling

src/app.py

@@ -7,7 +7,12 @@ import threading
 import gradio as gr
 from fastapi.responses import JSONResponse, RedirectResponse
 from fastapi import FastAPI, Request, Response, status
-from src.utils import manage_urls, wake_up_space, get_wake_up_action_name
+from src.utils import (
+    manage_urls,
+    wake_up_space,
+    get_wake_up_action_name,
+    get_url_list,
+)
 
 # Charger les variables d'environnement
 load_dotenv()
@@ -31,10 +36,6 @@ async def basic_auth(request: Request, call_next):
     
     return await call_next(request)
 
-# Liste d'URLs à pinger
-default_urls = os.getenv("DEFAULT_URLS", "")
-url_list = [url.strip() for url in default_urls.split(",") if url.strip()]
-
 # Interface Gradio
 interface = gr.Interface(
     fn=manage_urls,
@@ -57,12 +58,12 @@ def root_redirect():
 
 @app.get("/check_health")
 def check_health():
-    return JSONResponse(content={"status": "ok", "nb_urls": len(url_list)})
+    return JSONResponse(content={"status": "ok", "nb_urls": len(get_url_list())})
 
 # Background thread pour pinger les URLs toutes les heures
 def ping_loop():
     while True:
-        for url in url_list:
+        for url in get_url_list():
             try:
                 if get_wake_up_action_name(url):
                     print(f"[wake-up] 🚀 {url}")

src/utils.py

@@ -3,21 +3,53 @@ from typing import Optional
 from bs4 import BeautifulSoup
 import time
 import re
+import os
+from dotenv import load_dotenv
 
-from src.app import url_list
+load_dotenv()
+
+def get_url_list():
+    # Liste d'URLs à pinger
+    default_urls = os.getenv("DEFAULT_URLS", "")
+    url_list = [url.strip() for url in default_urls.split(",") if url.strip()]
+
+    return url_list
+
+def add_url(url: str) -> None:
+    """
+    Add a new URL to the list of URLs.
+    """
+    if url not in get_url_list():
+        os.environ["DEFAULT_URLS"] = ",".join([os.environ["DEFAULT_URLS"], url])
+        print(f"Added URL: {url}")
+    else:
+        print(f"URL already exists: {url}")
+
+def remove_url(url: str) -> None:
+    """
+    Remove a URL from the list of URLs.
+    """
+    url_list = get_url_list()
+    if url in url_list:
+        url_list.remove(url)
+        os.environ["DEFAULT_URLS"] = ",".join(url_list)
+        print(f"Removed URL: {url}")
+    else:
+        print(f"URL not found: {url}")
 
 # Fonction Gradio pour ajouter / retirer des URLs
 def manage_urls(action, url):
-    global url_list
+    url_list = get_url_list()
+
     if action == "Ajouter":
         if url and url not in url_list:
-            url_list.append(url)
+            add_url(url)
             message = f"✅ Ajouté: {url}"
         else:
             message = "⚠️ URL déjà présente ou vide"
     elif action == "Supprimer":
         if url in url_list:
-            url_list.remove(url)
+            remove_url(url)
             message = f"❌ Supprimé: {url}"
         else:
             message = "⚠️ URL non trouvée"
@@ -25,7 +57,7 @@ def manage_urls(action, url):
         message = "❓ Action invalide"
 
     # Toujours retourner la liste actuelle formatée
-    return message, "\n".join(url_list)
+    return message, "\n".join(get_url_list())
 
 def wake_up_space(space_url: str):
     action_name = get_wake_up_action_name(space_url)