[PR #24771] [CLOSED] refactor: remove unused GET /files/ endpoint #131498

New Issue

GiteaMirror · 2026-05-21T17:04:27-05:00

GiteaMirror commented

2026-05-21 17:04:27 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/24771
Author: @Classic298
Created: 5/15/2026
Status: ❌ Closed

Base: dev ← Head: remove-unused-files-list-endpoint

📝 Commits (1)

577a365 refactor: remove unused GET /files/ endpoint

📊 Changes

3 files changed (+0 additions, -85 deletions)

View changed files

📝 backend/open_webui/models/files.py (+0 -25)
📝 backend/open_webui/routers/files.py (+0 -29)
📝 src/lib/apis/files/index.ts (+0 -31)

📄 Description

GET /files/ returned a user's file list with full extracted file content inline by default (content=True), flagged as a High OOM risk in open-webui#22206. It was already paginated server-side (50 per page), but its only frontend wrapper (getFiles in src/lib/apis/files/index.ts) is dead: nothing imports or calls it anywhere in the codebase, and no other code path hits GET /files/. Rather than refactor an endpoint with no callers, remove it.

Drops with it the now-orphaned Files.get_file_list data-layer method and FileListResponse model (both used only by this endpoint), the now-unused PAGE_SIZE constant and FileListResponse import in the files router, and the dead getFiles frontend wrapper.

The paginated GET /files/search endpoint remains the supported way to enumerate a user's files.

Ref: open-webui#22206

Contributor License Agreement

By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.

Note

Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/open-webui/pull/24771 **Author:** [@Classic298](https://github.com/Classic298) **Created:** 5/15/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `remove-unused-files-list-endpoint` --- ### 📝 Commits (1) - [`577a365`](https://github.com/open-webui/open-webui/commit/577a36504244785aea033347714774228d456b68) refactor: remove unused GET /files/ endpoint ### 📊 Changes **3 files changed** (+0 additions, -85 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/models/files.py` (+0 -25) 📝 `backend/open_webui/routers/files.py` (+0 -29) 📝 `src/lib/apis/files/index.ts` (+0 -31) </details> ### 📄 Description GET /files/ returned a user's file list with full extracted file content inline by default (content=True), flagged as a High OOM risk in open-webui#22206. It was already paginated server-side (50 per page), but its only frontend wrapper (getFiles in src/lib/apis/files/index.ts) is dead: nothing imports or calls it anywhere in the codebase, and no other code path hits GET /files/. Rather than refactor an endpoint with no callers, remove it. Drops with it the now-orphaned Files.get_file_list data-layer method and FileListResponse model (both used only by this endpoint), the now-unused PAGE_SIZE constant and FileListResponse import in the files router, and the dead getFiles frontend wrapper. The paginated GET /files/search endpoint remains the supported way to enumerate a user's files. Ref: open-webui#22206 ### Contributor License Agreement  - [x] By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. > [!NOTE] > Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-05-21 17:04:27 -05:00

GiteaMirror closed this issue

2026-05-21 17:04:31 -05:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/open-webui#131498