[PR #4107] [CLOSED] types/model: make Name.Filepath substitute colons in host with ("%") #58079

Closed
opened 2026-04-29 12:54:18 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/4107
Author: @bmizerany
Created: 5/2/2024
Status: Closed

Base: mainHead: bmizerany/filepathwithcoloninhost


📝 Commits (1)

  • 61b287c types/model: make Name.Filepath substitute colons in host with ("%")

📊 Changes

5 files changed (+112 additions, -5 deletions)

View changed files

📝 server/fixblobs.go (+42 -0)
📝 server/fixblobs_test.go (+49 -0)
📝 server/routes.go (+7 -0)
📝 types/model/name.go (+2 -2)
📝 types/model/name_test.go (+12 -3)

📄 Description

This makes the filepath legal on all supported platforms.

Fixes #4088


🔄 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/4107 **Author:** [@bmizerany](https://github.com/bmizerany) **Created:** 5/2/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `bmizerany/filepathwithcoloninhost` --- ### 📝 Commits (1) - [`61b287c`](https://github.com/ollama/ollama/commit/61b287cf25ca033a29cf02a7f97f7fe31483c783) types/model: make Name.Filepath substitute colons in host with ("%") ### 📊 Changes **5 files changed** (+112 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `server/fixblobs.go` (+42 -0) 📝 `server/fixblobs_test.go` (+49 -0) 📝 `server/routes.go` (+7 -0) 📝 `types/model/name.go` (+2 -2) 📝 `types/model/name_test.go` (+12 -3) </details> ### 📄 Description This makes the filepath legal on all supported platforms. Fixes #4088 --- <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 2026-04-29 12:54:18 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#58079