issue: Serply params passed incorrectly in web_search #5998

Closed
opened 2025-11-11 16:41:54 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @jsweetzer-ea on GitHub (Aug 8, 2025).

Check Existing Issues

  • I have searched the existing issues and discussions.
  • I am using the latest version of Open WebUI.

Installation Method

Git Clone

Open WebUI Version

v0.6.18

Ollama Version (if applicable)

No response

Operating System

AWS linux

Browser (if applicable)

No response

Confirmation

  • I have read and followed all instructions in README.md.
  • I am using the latest version of both Open WebUI and Ollama.
  • I have included the browser console logs.
  • I have included the Docker container logs.
  • I have provided every relevant configuration, setting, and environment variable used in my setup.
  • I have clearly listed every relevant configuration, custom setting, environment variable, and command-line option that influences my setup (such as Docker Compose overrides, .env values, browser settings, authentication configurations, etc).
  • I have documented step-by-step reproduction instructions that are precise, sequential, and leave nothing to interpretation. My steps:
  • Start with the initial platform/version/OS and dependencies used,
  • Specify exact install/launch/configure commands,
  • List URLs visited, user input (incl. example values/emails/passwords if needed),
  • Describe all options and toggles enabled or changed,
  • Include any files or environmental changes,
  • Identify the expected and actual result at each stage,
  • Ensure any reasonably skilled user can follow and hit the same issue.

Expected Behavior

When Serply is set as web search provider and a valid API key is provided, it should function with the web search tool.

Actual Behavior

Serply integration is broken

search_serply positional args are here
https://github.com/open-webui/open-webui/blob/main/backend/open_webui/retrieval/web/serply.py#L13-L22

when args are passed to the function here as part of web_search
https://github.com/open-webui/open-webui/blob/main/backend/open_webui/routers/retrieval.py#L1782-L1788

the 4th param passed is the domain filter list, and it is expected to be the "hl" param.

Steps to Reproduce

Set Serply as web provider, set Serply API key, try to use as Web Search tool in LLM chat

Logs & Screenshots

Image

https://github.com/open-webui/open-webui/blob/main/backend/open_webui/retrieval/web/serply.py#L40

error originates when "lower()" is called on the list of domains to filter

Additional Information

No response

Originally created by @jsweetzer-ea on GitHub (Aug 8, 2025). ### Check Existing Issues - [x] I have searched the existing issues and discussions. - [x] I am using the latest version of Open WebUI. ### Installation Method Git Clone ### Open WebUI Version v0.6.18 ### Ollama Version (if applicable) _No response_ ### Operating System AWS linux ### Browser (if applicable) _No response_ ### Confirmation - [x] I have read and followed all instructions in `README.md`. - [x] I am using the latest version of **both** Open WebUI and Ollama. - [x] I have included the browser console logs. - [x] I have included the Docker container logs. - [x] I have **provided every relevant configuration, setting, and environment variable used in my setup.** - [x] I have clearly **listed every relevant configuration, custom setting, environment variable, and command-line option that influences my setup** (such as Docker Compose overrides, .env values, browser settings, authentication configurations, etc). - [x] I have documented **step-by-step reproduction instructions that are precise, sequential, and leave nothing to interpretation**. My steps: - Start with the initial platform/version/OS and dependencies used, - Specify exact install/launch/configure commands, - List URLs visited, user input (incl. example values/emails/passwords if needed), - Describe all options and toggles enabled or changed, - Include any files or environmental changes, - Identify the expected and actual result at each stage, - Ensure any reasonably skilled user can follow and hit the same issue. ### Expected Behavior When Serply is set as web search provider and a valid API key is provided, it should function with the web search tool. ### Actual Behavior Serply integration is broken search_serply positional args are here https://github.com/open-webui/open-webui/blob/main/backend/open_webui/retrieval/web/serply.py#L13-L22 when args are passed to the function here as part of web_search https://github.com/open-webui/open-webui/blob/main/backend/open_webui/routers/retrieval.py#L1782-L1788 the 4th param passed is the domain filter list, and it is expected to be the "hl" param. ### Steps to Reproduce Set Serply as web provider, set Serply API key, try to use as Web Search tool in LLM chat ### Logs & Screenshots <img width="1498" height="260" alt="Image" src="https://github.com/user-attachments/assets/ecb5463a-3970-42b0-a06b-47ce881bd39d" /> https://github.com/open-webui/open-webui/blob/main/backend/open_webui/retrieval/web/serply.py#L40 error originates when "lower()" is called on the list of domains to filter ### Additional Information _No response_
GiteaMirror added the bug label 2025-11-11 16:41:54 -06:00
Author
Owner

@tjbck commented on GitHub (Aug 8, 2025):

Addressed with e8cb57750b41452c84c57953263cb627479ee2ed!

@tjbck commented on GitHub (Aug 8, 2025): Addressed with e8cb57750b41452c84c57953263cb627479ee2ed!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#5998