fix : setting page issues

main.py

@@ -169,6 +169,8 @@ DEFAULT_PC_KEY  = os.getenv("DEFAULT_PINECONE_KEY",  "")
 DEFAULT_CLD_URL = os.getenv("DEFAULT_CLOUDINARY_URL", "")
 
 def _is_default_key(key: str, default: str) -> bool:
+    if not key or not key.strip():   # empty key never matches — prevents blocking users with no keys
+        return False
     return bool(default) and key.strip() == default.strip()
 
 
@@ -991,34 +993,39 @@ async def delete_account(
 
     creds = get_cloudinary_creds(user_cloudinary_url)
 
-    # ── Cloudinary: paginated delete ALL resources then folders ────
-    try:
-        deleted = await asyncio.to_thread(_cld_delete_all_paginated, creds)
-        _log_fn("INFO", f"Account delete Cloudinary: {deleted} resources removed")
-    except Exception as e:
-        _log_fn("WARNING", f"Account delete Cloudinary: {e}")
-
-    try:
-        folders_res = await asyncio.to_thread(_cld_root_folders, creds)
-        folder_tasks = [
-            asyncio.to_thread(_cld_remove_folder, f["name"], creds)
-            for f in folders_res.get("folders", [])
-        ]
-        if folder_tasks:
-            await asyncio.gather(*folder_tasks, return_exceptions=True)
-    except Exception as e:
-        _log_fn("WARNING", f"Account delete Cloudinary folders: {e}")
+    # ── Cloudinary: skip gracefully if user has no credentials ───
+    if creds.get("cloud_name"):
+        try:
+            deleted = await asyncio.to_thread(_cld_delete_all_paginated, creds)
+            _log_fn("INFO", f"Account delete Cloudinary: {deleted} resources removed")
+        except Exception as e:
+            _log_fn("WARNING", f"Account delete Cloudinary: {e}")
+        try:
+            folders_res = await asyncio.to_thread(_cld_root_folders, creds)
+            folder_tasks = [
+                asyncio.to_thread(_cld_remove_folder, f["name"], creds)
+                for f in folders_res.get("folders", [])
+            ]
+            if folder_tasks:
+                await asyncio.gather(*folder_tasks, return_exceptions=True)
+        except Exception as e:
+            _log_fn("WARNING", f"Account delete Cloudinary folders: {e}")
+    else:
+        _log_fn("INFO", "Account delete: no Cloudinary credentials — skipping CDN wipe")
 
-    # ── Pinecone: delete both indexes ────────────────────────────
-    try:
-        pc       = _get_pinecone(user_pinecone_key)
-        existing = {idx.name for idx in await asyncio.to_thread(pc.list_indexes)}
-        tasks    = []
-        if IDX_OBJECTS in existing: tasks.append(asyncio.to_thread(pc.delete_index, IDX_OBJECTS))
-        if IDX_FACES   in existing: tasks.append(asyncio.to_thread(pc.delete_index, IDX_FACES))
-        if tasks: await asyncio.gather(*tasks)
-    except Exception as e:
-        _log_fn("WARNING", f"Account delete Pinecone: {e}")
+    # ── Pinecone: skip gracefully if user has no key ─────────────
+    if user_pinecone_key and user_pinecone_key.strip():
+        try:
+            pc       = _get_pinecone(user_pinecone_key)
+            existing = {idx.name for idx in await asyncio.to_thread(pc.list_indexes)}
+            tasks    = []
+            if IDX_OBJECTS in existing: tasks.append(asyncio.to_thread(pc.delete_index, IDX_OBJECTS))
+            if IDX_FACES   in existing: tasks.append(asyncio.to_thread(pc.delete_index, IDX_FACES))
+            if tasks: await asyncio.gather(*tasks)
+        except Exception as e:
+            _log_fn("WARNING", f"Account delete Pinecone: {e}")
+    else:
+        _log_fn("INFO", "Account delete: no Pinecone key — skipping vector wipe")
 
     log("WARNING", "danger.delete_account.complete",
         user_id=user_id or "anonymous", ip=ip,