[PR #916] [MERGED] fix(client): trim trailing slash #9123

Closed
opened 2025-11-12 14:59:40 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/916
Author: @mxyng
Created: 10/26/2023
Status: Merged
Merged: 10/26/2023
Merged by: @mxyng

Base: mainHead: mxyng/fix-client-host


📝 Commits (1)

  • 28c3f28 client: fix trailing slash

📊 Changes

2 files changed (+55 additions, -2 deletions)

View changed files

📝 api/client.go (+12 -2)
api/client_test.go (+43 -0)

📄 Description

Also sets different default ports when scheme is in OLLAMA_HOST and is either http or https.

e.g. OLLAMA_HOST=https://example.com will now use port 443 instead of 11434.

Resolves #910


🔄 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/ollama/ollama/pull/916 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 10/26/2023 **Status:** ✅ Merged **Merged:** 10/26/2023 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/fix-client-host` --- ### 📝 Commits (1) - [`28c3f28`](https://github.com/ollama/ollama/commit/28c3f288e2fedbeaabc8b67f04371b743cb74f62) client: fix trailing slash ### 📊 Changes **2 files changed** (+55 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `api/client.go` (+12 -2) ➕ `api/client_test.go` (+43 -0) </details> ### 📄 Description Also sets different default ports when scheme is in OLLAMA_HOST and is either `http` or `https`. e.g. `OLLAMA_HOST=https://example.com` will now use port 443 instead of 11434. Resolves #910 --- <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-12 14:59:40 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama-ollama#9123