logout route deleting cookie fix (#7886)

* logout button now deletes cookies by calling RedirectResponse before deleting cookies

* add changeset

* response

---------

Co-authored-by: gradio-pr-bot <gradio-pr-bot@users.noreply.github.com>
Co-authored-by: Abubakar Abid <abubakar@huggingface.co>

.changeset/angry-towns-mate.md

@@ -0,0 +1,5 @@
+---
+"gradio": patch
+---
+
+fix:logout route deleting cookie fix

gradio/routes.py

@@ -37,7 +37,7 @@
 import httpx
 import markupsafe
 import orjson
-from fastapi import BackgroundTasks, Depends, FastAPI, HTTPException, Response, status
+from fastapi import BackgroundTasks, Depends, FastAPI, HTTPException, status
 from fastapi.responses import (
     FileResponse,
     HTMLResponse,
@@ -331,7 +331,8 @@ def login(form_data: OAuth2PasswordRequestForm = Depends()):
         else:
 
             @app.get("/logout")
-            def logout(response: Response, user: str = Depends(get_current_user)):
+            def logout(user: str = Depends(get_current_user)):
+                response = RedirectResponse(url="/", status_code=status.HTTP_302_FOUND)
                 response.delete_cookie(key=f"access-token-{app.cookie_id}", path="/")
                 response.delete_cookie(
                     key=f"access-token-unsecure-{app.cookie_id}", path="/"
@@ -340,7 +341,7 @@ def logout(response: Response, user: str = Depends(get_current_user)):
                 for token in list(app.tokens.keys()):
                     if app.tokens[token] == user:
                         del app.tokens[token]
-                return RedirectResponse(url="/", status_code=status.HTTP_302_FOUND)
+                return response
 
         ###############
         # Main Routes