[PR #73] [MERGED] Fix OpenAI test endpoint to preserve /v1 base path #87

New issue

Closed

opened 2026-02-27 16:42:30 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 16:42:30 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/router-for-me/Cli-Proxy-API-Management-Center/pull/73
Author: @LTbinglingfeng
Created: 1/29/2026
Status: ✅ Merged
Merged: 1/29/2026
Merged by: @LTbinglingfeng

Base: dev ← Head: codex/fix-base-url-handling-for-openai-interface

📝 Commits (1)

ab86fcf Fix OpenAI test endpoint base URL

📊 Changes

1 file changed (+0 additions, -3 deletions)

View changed files

📝 src/components/providers/utils.ts (+0 -3)

📄 Description

Motivation

The OpenAI provider test logic removed a trailing /v1 from the configured Base URL, which caused test requests to target the wrong path and return 404 instead of appending the request path to the user-provided URL.

Description

Removed the special-case branch in buildOpenAIChatCompletionsEndpoint so it no longer strips /v1 and now always appends /chat/completions to the normalized base URL in src/components/providers/utils.ts.

Testing

No automated tests were run for this change.

Codex Task

_{🔄 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/router-for-me/Cli-Proxy-API-Management-Center/pull/73 **Author:** [@LTbinglingfeng](https://github.com/LTbinglingfeng) **Created:** 1/29/2026 **Status:** ✅ Merged **Merged:** 1/29/2026 **Merged by:** [@LTbinglingfeng](https://github.com/LTbinglingfeng) **Base:** `dev` ← **Head:** `codex/fix-base-url-handling-for-openai-interface` --- ### 📝 Commits (1) - [`ab86fcf`](https://github.com/router-for-me/Cli-Proxy-API-Management-Center/commit/ab86fcf674f7265e845591c98b35459de8a4ac01) Fix OpenAI test endpoint base URL ### 📊 Changes **1 file changed** (+0 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `src/components/providers/utils.ts` (+0 -3) </details> ### 📄 Description ### Motivation - The OpenAI provider test logic removed a trailing `/v1` from the configured Base URL, which caused test requests to target the wrong path and return 404 instead of appending the request path to the user-provided URL. ### Description - Removed the special-case branch in `buildOpenAIChatCompletionsEndpoint` so it no longer strips `/v1` and now always appends `/chat/completions` to the normalized base URL in `src/components/providers/utils.ts`. ### Testing - No automated tests were run for this change. ------ [Codex Task](https://chatgpt.com/codex/tasks/task_b_697b4a27a38c8327a83b5958ef68469d) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-27 16:42:30 +03:00

closed this issue
added the
pull-request
label

No labels

enhancement

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/Cli-Proxy-API-Management-Center#87

No description provided.

Rows
Columns