mirror of
https://github.com/open-webui/open-webui.git
synced 2026-05-06 02:48:13 -05:00
[PR #15632] [MERGED] fix: don't over quote forwarded headers #23896
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/15632
Author: @athoik
Created: 7/10/2025
Status: ✅ Merged
Merged: 7/11/2025
Merged by: @tjbck
Base:
dev← Head:quote📝 Commits (1)
9675817fix: don't over quote forwarded headers📊 Changes
5 files changed (+84 additions, -84 deletions)
View changed files
📝
backend/open_webui/retrieval/utils.py(+12 -12)📝
backend/open_webui/routers/audio.py(+4 -4)📝
backend/open_webui/routers/images.py(+4 -4)📝
backend/open_webui/routers/ollama.py(+36 -36)📝
backend/open_webui/routers/openai.py(+28 -28)📄 Description
Changelog Entry
Description
Fix introduced on #15035 is over quoting headers.
Eg mails instead of user@example.com shown as user%40example.com Eg names instead of First Last shown as First%20Last
Also we are spending some time quoting ids and roles without required.
Keep quote only on user name, initially had problem based on the discussion https://github.com/open-webui/open-webui/discussions/14391
Also add space in safe characters, in order remove %20 from names.
Breaking Changes
Screenshots or Videos
This commit will fix wrong %20 inserted on names and %40 insertead on emails.
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.