From 624b360eb19593ed08d87f030b9df0b75b6258c0 Mon Sep 17 00:00:00 2001
From: DrMelone <27028174+Classic298@users.noreply.github.com>
Date: Mon, 13 Apr 2026 00:26:43 +0200
Subject: [PATCH] fix: move upload_and_replace route decorator to the actual
 handler

The previous rollback-helper extraction landed the decorator above
_rollback_new_file instead of upload_and_replace_file, which left the
POST /{id}/file/upload_and_replace route pointed at an internal helper
with the wrong signature (no id path param, no UploadFile, no
old_file_id form) and the real handler unexposed. Move the decorator
back to upload_and_replace_file and keep _rollback_new_file as a plain
private helper.
---
 backend/open_webui/routers/knowledge.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/open_webui/routers/knowledge.py b/backend/open_webui/routers/knowledge.py
index 85abc8d58a..d06075e5ed 100644
--- a/backend/open_webui/routers/knowledge.py
+++ b/backend/open_webui/routers/knowledge.py
@@ -701,7 +701,6 @@ class UploadAndReplaceResponse(BaseModel):
     filename: str
 
 
-@router.post('/{id}/file/upload_and_replace', response_model=UploadAndReplaceResponse)
 async def _rollback_new_file(
     kb_id: str,
     new_file_id: str,
@@ -727,6 +726,7 @@ async def _rollback_new_file(
         )
 
 
+@router.post('/{id}/file/upload_and_replace', response_model=UploadAndReplaceResponse)
 async def upload_and_replace_file(
     request: Request,
     id: str,