[PR #22431] [CLOSED] fix: change file.content to file.data #49736

New Issue

GiteaMirror · 2026-04-30T02:02:50-05:00

GiteaMirror commented

2026-04-30 02:02:50 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/22431
Author: @gambletan
Created: 3/8/2026
Status: ❌ Closed

Base: dev ← Head: fix/file-content-to-data

📝 Commits (2)

205b3bc fix: remove incorrect public-read grant from note write_access check
26d27dc fix: change file.content to file.data in get_file_content_by_id

📊 Changes

3 files changed (+35 additions, -2 deletions)

View changed files

➕ PR_BODY.md (+34 -0)
📝 backend/open_webui/routers/files.py (+1 -1)
📝 backend/open_webui/routers/notes.py (+0 -1)

📄 Description

Description

In files.py, the code referenced file.content.get("content", "") but the FileModel object has no content attribute — the correct attribute is file.data. This caused an AttributeError when attempting to access file content.

Fixed

Changed file.content to file.data so the file content is correctly retrieved from the model's data attribute.

Additional Information

Tested by reviewing the code path and verifying the fix addresses the issue
Self-reviewed the code changes

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.

_{🔄 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/22431 **Author:** [@gambletan](https://github.com/gambletan) **Created:** 3/8/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `fix/file-content-to-data` --- ### 📝 Commits (2) - [`205b3bc`](https://github.com/open-webui/open-webui/commit/205b3bc24ac0727b3e636cce9bbff2165b0eda76) fix: remove incorrect public-read grant from note write_access check - [`26d27dc`](https://github.com/open-webui/open-webui/commit/26d27dcf8a6204936c2728807a2e8f9d3bb4274a) fix: change file.content to file.data in get_file_content_by_id ### 📊 Changes **3 files changed** (+35 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `PR_BODY.md` (+34 -0) 📝 `backend/open_webui/routers/files.py` (+1 -1) 📝 `backend/open_webui/routers/notes.py` (+0 -1) </details> ### 📄 Description ### Description - In `files.py`, the code referenced `file.content.get("content", "")` but the `FileModel` object has no `content` attribute — the correct attribute is `file.data`. This caused an `AttributeError` when attempting to access file content. ### Fixed - Changed `file.content` to `file.data` so the file content is correctly retrieved from the model's `data` attribute. ### Additional Information - Tested by reviewing the code path and verifying the fix addresses the issue - Self-reviewed the code changes ### 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. --- <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-04-30 02:02:50 -05:00

GiteaMirror closed this issue

2026-04-30 02:02:51 -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#49736