Title: fix: Remove trailing slash from OLLAMA_BASE_URL to prevent 404 errors
Target Branch: dev
Description
This pull request addresses an issue where the trailing slash in the OLLAMA_BASE_URL configuration was causing 404 errors when making HTTP requests. By using Wireshark to capture the HTTP traffic, I identified that the extra slash was the root cause of the problem.
The fix involves removing the trailing slash from OLLAMA_BASE_URL if it exists, which ensures that the URL is correctly formatted and avoids the 404 error.
Changelog Entry
Fixed: Trailing slash in OLLAMA_BASE_URL causing 404 errors
Testing
Manual Testing: I manually tested the Docker command with the corrected OLLAMA_BASE_URL and confirmed that the 404 error no longer occurs.
Code Review
Self-Review: I have performed a self-review of the changes and ensured adherence to the project's coding standards.
Documentation
Updated: No documentation updates are required as this fix is internal to the configuration handling.
Dependencies
No New Dependencies: This fix does not introduce any new dependencies.
Thank you for reviewing this pull request. Your feedback is highly appreciated.
🔄 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/7491
**Author:** [@hi-unc1e](https://github.com/hi-unc1e)
**Created:** 11/30/2024
**Status:** ❌ Closed
**Base:** `dev` ← **Head:** `main`
---
### 📝 Commits (1)
- [`c8ef02e`](https://github.com/open-webui/open-webui/commit/c8ef02e6f8eafe15772813e4a52b57da0f327970) Update config.py - remove ending slash to avoid 404 error
### 📊 Changes
**1 file changed** (+2 additions, -0 deletions)
<details>
<summary>View changed files</summary>
📝 `backend/open_webui/config.py` (+2 -0)
</details>
### 📄 Description
### Pull Request Description
**Title:** fix: Remove trailing slash from OLLAMA_BASE_URL to prevent 404 errors
**Target Branch:** dev
### Description
This pull request addresses an issue where the trailing slash in the `OLLAMA_BASE_URL` configuration was causing 404 errors when making HTTP requests. By using Wireshark to capture the HTTP traffic, I identified that the extra slash was the root cause of the problem.
The fix involves removing the trailing slash from `OLLAMA_BASE_URL` if it exists, which ensures that the URL is correctly formatted and avoids the 404 error.
### Changelog Entry
- **Fixed:** Trailing slash in `OLLAMA_BASE_URL` causing 404 errors
### Testing
- **Manual Testing:** I manually tested the Docker command with the corrected `OLLAMA_BASE_URL` and confirmed that the 404 error no longer occurs.
### Code Review
- **Self-Review:** I have performed a self-review of the changes and ensured adherence to the project's coding standards.
### Documentation
- **Updated:** No documentation updates are required as this fix is internal to the configuration handling.
### Dependencies
- **No New Dependencies:** This fix does not introduce any new dependencies.
### Related Discussion
- **Discussion:** [Discussions#6065](https://github.com/open-webui/open-webui/discussions/6065)
---
Thank you for reviewing this pull request. Your feedback is highly appreciated.
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/7491
Author: @hi-unc1e
Created: 11/30/2024
Status: ❌ Closed
Base:
dev← Head:main📝 Commits (1)
c8ef02eUpdate config.py - remove ending slash to avoid 404 error📊 Changes
1 file changed (+2 additions, -0 deletions)
View changed files
📝
backend/open_webui/config.py(+2 -0)📄 Description
Pull Request Description
Title: fix: Remove trailing slash from OLLAMA_BASE_URL to prevent 404 errors
Target Branch: dev
Description
This pull request addresses an issue where the trailing slash in the
OLLAMA_BASE_URLconfiguration was causing 404 errors when making HTTP requests. By using Wireshark to capture the HTTP traffic, I identified that the extra slash was the root cause of the problem.The fix involves removing the trailing slash from
OLLAMA_BASE_URLif it exists, which ensures that the URL is correctly formatted and avoids the 404 error.Changelog Entry
OLLAMA_BASE_URLcausing 404 errorsTesting
OLLAMA_BASE_URLand confirmed that the 404 error no longer occurs.Code Review
Documentation
Dependencies
Related Discussion
Thank you for reviewing this pull request. Your feedback is highly appreciated.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.