mirror of
https://github.com/open-webui/open-webui.git
synced 2026-07-16 06:03:26 -05:00
[GH-ISSUE #1478] Helm chart adds extra / to end of ollama URL, which is invalid #132240
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/open-webui#132240
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?
Originally created by @cprivitere on GitHub (Apr 9, 2024).
Original GitHub issue: https://github.com/open-webui/open-webui/issues/1478
Bug Report
Description
Bug Summary:
This is a follow up from #1422. That got us closer to fixing the issue, but it's still present because a
/was left behind. This is just to ask that that last/be removed. It seems unlike most browsers, the http libraries open-webui uses doesn't like having two/in a row in a URL.Fix should be a quick removal of the trailing
/from line 10 of helpers.tpl.Helm chart adds extra / to end of ollama URL, which is invalid. Can by fixed by removing /from line 10 of the _helpers function
Steps to Reproduce:
Install the helm chart and log in and check ollama's status in the settings. Just trim off the / and then refresh and it works.
Expected Behavior:
Should just work without having to make this fix.
Actual Behavior:
Can't connect to ollama and thus no models work, nor can they be downloaded.
Environment
Linux (kubernetes on Ubuntu 20.04)
Microsoft Edge
Reproduction Details
Confirmation:
Logs and Screenshots
Installation Method
Helm
@tjbck commented on GitHub (Apr 10, 2024):
Feel free to make a PR!
@dariothornhill commented on GitHub (Apr 15, 2024):
I missed the the issue linking in the PR. I believe this can be closed now.