[PR #15204] [MERGED] app: use the same client for inference and other requests #25612

Closed
opened 2026-04-19 18:18:34 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/15204
Author: @drifkin
Created: 4/2/2026
Status: Merged
Merged: 4/2/2026
Merged by: @drifkin

Base: mainHead: drifkin/app-consistent-client


📝 Commits (1)

  • 68ce1a4 app: use the same client for inference and other requests

📊 Changes

2 files changed (+40 additions, -7 deletions)

View changed files

📝 app/ui/ui.go (+12 -7)
📝 app/ui/ui_test.go (+28 -0)

📄 Description

Previously we were accidentally using different clients/UAs depending on whether it was an inference call or a different call. This change makes them consistent, other than the timeout being different.


🔄 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/15204 **Author:** [@drifkin](https://github.com/drifkin) **Created:** 4/2/2026 **Status:** ✅ Merged **Merged:** 4/2/2026 **Merged by:** [@drifkin](https://github.com/drifkin) **Base:** `main` ← **Head:** `drifkin/app-consistent-client` --- ### 📝 Commits (1) - [`68ce1a4`](https://github.com/ollama/ollama/commit/68ce1a48a9ce238e611971d111b9264a13a301bd) app: use the same client for inference and other requests ### 📊 Changes **2 files changed** (+40 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `app/ui/ui.go` (+12 -7) 📝 `app/ui/ui_test.go` (+28 -0) </details> ### 📄 Description Previously we were accidentally using different clients/UAs depending on whether it was an inference call or a different call. This change makes them consistent, other than the timeout being different. --- <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-19 18:18:34 -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#25612