Enhance CORS configuration and improve item upload handling

lenny/app.py

@@ -18,7 +18,11 @@
     CORSMiddleware,
     allow_origins=[
         "http://localhost:3000",
-        "http://127.0.0.1:3000"
+        "http://127.0.0.1:3000",
+        "http://localhost:3001",
+        "http://127.0.0.1:3001",
+        "http://localhost:3002",
+
     ],
     allow_credentials=True,
     allow_methods=["*"],

lenny/core/api.py

@@ -230,7 +230,7 @@ def _resolve_ip_to_hostname(cls, client_ip: str) -> str:
 
     @classmethod
     def is_allowed_uploader(cls, client_ip: str) -> bool:
-        if client_ip in ("127.0.0.1", "::1"):
+        if client_ip in ("127.0.0.1", "::1") or client_ip.startswith("172.18.0."): # localhost and docker internal
             return True
 
         if host := cls._resolve_ip_to_hostname(client_ip):

lenny/routes/api.py

@@ -189,7 +189,13 @@ async def upload(
 ):
 
     try:
-        item = LennyAPI.add(
+        if item := Item.exists(openlibrary_edition):
+            return HTMLResponse(
+                status_code=status.HTTP_409_CONFLICT,
+                content=f"Item with OpenLibrary Edition ID {openlibrary_edition} already exists."
+            )
+        if not item:
+            LennyAPI.add(
             openlibrary_edition=openlibrary_edition,
             files=[file],  # TODO expand to allow multiple
             uploader_ip=request.client.host,
@@ -201,8 +207,6 @@ async def upload(
         )
     except UploaderNotAllowedError as e:
         raise HTTPException(status_code=503, details=str(e))
-    except ItemExistsError as e:
-        raise HTTPException(status_code=409, detail=str(e))
     except InvalidFileError as e:
         raise HTTPException(status_code=400, detail=str(e))
     except DatabaseInsertError as e: