[PR #16381] [MERGED] fix: failure to bind metadata variable on insert for PGVECTOR_PGCRYPTO feature returning syntax error #10912

New Issue

GiteaMirror · 2025-11-11T19:17:02-06:00

GiteaMirror commented

2025-11-11 19:17:02 -06:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/16381
Author: @psy42a
Created: 8/8/2025
Status: ✅ Merged
Merged: 8/8/2025
Merged by: @tjbck

Base: dev ← Head: patch-1

📝 Commits (1)

f3b0f7d Fix syntax error where the previous use of :metadata::text in some sqlachamy/postgres versions doesn't bind at all

📊 Changes

1 file changed (+7 additions, -4 deletions)

View changed files

📝 backend/open_webui/retrieval/vector/dbs/pgvector.py (+7 -4)

📄 Description

with CLA this time

In testing of the experimental PGVECTOR_PGCRYPTO feature we found that it failed to process/upload any documents due to a syntax error raised from failure to bind the metadata variable in the form "pgp_sym_encrypt(:metadata::text, :key)" this proposed fix (potentially just removing ::text might also work) explicitly serialises the json as a text variable and ensures that sqlalchemy correctly treats it as text and binds it correctly. Tested in Openwebui version v0.6.18 using Postgres 17.4

3 changes were made to the insert / upsert functions:

Added json_metadata = json.dumps(item["metadata"]) as seperate line
updated reference where this field is passed into sqlalchemy to "metadata_text": json_metadata,
and updated the pgp_sym_encrypt(:metadata_text, :key) line accordingly in the insert and upsert functions
We have then validated upload was successful for the document_chunk table with text, vmetadata fields defined as bytea fields and validated document retrieval and image summary of pdf works as expected using docling and the image summary api functionality.

Fixed
Corrected PGVECTOR_PGCRYPTO implementation to remove syntax error relating to metadata field

Contributor License Agreement
By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreements (CLA) and Open WebUI specific Contributor License Agreements, 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/16381 **Author:** [@psy42a](https://github.com/psy42a) **Created:** 8/8/2025 **Status:** ✅ Merged **Merged:** 8/8/2025 **Merged by:** [@tjbck](https://github.com/tjbck) **Base:** `dev` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`f3b0f7d`](https://github.com/open-webui/open-webui/commit/f3b0f7d3583e2bfa0f186129056275276146958f) Fix syntax error where the previous use of :metadata::text in some sqlachamy/postgres versions doesn't bind at all ### 📊 Changes **1 file changed** (+7 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/retrieval/vector/dbs/pgvector.py` (+7 -4) </details> ### 📄 Description with CLA this time In testing of the experimental PGVECTOR_PGCRYPTO feature we found that it failed to process/upload any documents due to a syntax error raised from failure to bind the metadata variable in the form "pgp_sym_encrypt(:metadata::text, :key)" this proposed fix (potentially just removing ::text might also work) explicitly serialises the json as a text variable and ensures that sqlalchemy correctly treats it as text and binds it correctly. Tested in Openwebui version v0.6.18 using Postgres 17.4 3 changes were made to the insert / upsert functions: Added json_metadata = json.dumps(item["metadata"]) as seperate line updated reference where this field is passed into sqlalchemy to "metadata_text": json_metadata, and updated the pgp_sym_encrypt(:metadata_text, :key) line accordingly in the insert and upsert functions We have then validated upload was successful for the document_chunk table with text, vmetadata fields defined as bytea fields and validated document retrieval and image summary of pdf works as expected using docling and the image summary api functionality. Fixed Corrected PGVECTOR_PGCRYPTO implementation to remove syntax error relating to metadata field Contributor License Agreement By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreements (CLA)](https://github.com/CONTRIBUTOR_LICENSE_AGREEMENT) and [Open WebUI specific Contributor License Agreements](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 2025-11-11 19:17:02 -06:00

GiteaMirror closed this issue

2025-11-11 19:17:02 -06:00

GiteaMirror referenced this issue

2026-04-19 22:06:32 -05:00

[GH-ISSUE #10887] 500: Internal Error #16065

GiteaMirror referenced this issue

2026-04-19 22:07:10 -05:00

[GH-ISSUE #10912] "500: Internal Error" after upgrading docker container from v0.5.16 to v0.5.17 #16078

GiteaMirror referenced this issue

2026-04-25 05:30:38 -05:00

[GH-ISSUE #10887] 500: Internal Error #31594