seewe/sub2api
1
0
Fork 0
mirror of https://github.com/Wei-Shaw/sub2api.git synced 2026-10-14 12:02:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #7822 from neotan/fix/issue-7670-bare-api-alias-bypass

Browse Source 
fix(web): bypass bare API aliases from embedded frontend fallback
This commit is contained in:
Wesley Liddick
2026-10-08 11:33:58 +08:00
committed by GitHub
parent bc63bd2874 abdfbb704c
commit 7ff5cff1bb
2 changed files with 56 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/internal/web/embed_on.go
+16
View File
@@ -362,10 +362,26 @@ func shouldBypassEmbeddedFrontend(path string) bool {
strings.HasPrefix(trimmed, "/setup/") ||
trimmed == "/health" ||
trimmed == "/models" ||
strings.HasPrefix(trimmed, "/models/") ||
trimmed == "/responses" ||
strings.HasPrefix(trimmed, "/responses/") ||
trimmed == "/chat/completions" ||
trimmed == "/embeddings" ||
trimmed == "/messages/count_tokens" ||
trimmed == "/alpha/search" ||
trimmed == "/web_search" ||
trimmed == "/x_search" ||
trimmed == "/tts" ||
trimmed == "/stt" ||
trimmed == "/realtime" ||
trimmed == "/custom-voices" ||
strings.HasPrefix(trimmed, "/custom-voices/") ||
trimmed == "/contents/generations/tasks" ||
strings.HasPrefix(trimmed, "/contents/generations/tasks/") ||
trimmed == "/v3/contents/generations/tasks" ||
strings.HasPrefix(trimmed, "/v3/contents/generations/tasks/") ||
strings.HasPrefix(trimmed, "/images/") ||
trimmed == "/videos" ||
strings.HasPrefix(trimmed, "/videos/")
}
backend/internal/web/embed_test.go
+40
View File
@@ -530,6 +530,9 @@ func TestFrontendServer_Middleware(t *testing.T) {
"/health",
"/responses",
"/responses/compact",
"/chat/completions",
"/models/gpt-5.5",
"/v3/contents/generations/tasks/task-123",
}
for _, path := range apiPaths {
@@ -689,6 +692,40 @@ func TestEmbeddedFrontendBypassesBareVideoAPIRoutes(t *testing.T) {
}
}
func TestEmbeddedFrontendBypassesBareAPIAliases(t *testing.T) {
for _, path := range []string{
"/chat/completions",
"/embeddings",
"/messages/count_tokens",
"/models/gpt-5.5",
"/videos",
"/tts",
"/stt",
"/custom-voices",
"/custom-voices/voice-123",
"/custom-voices/voice-123/audio",
"/realtime",
"/web_search",
"/x_search",
"/contents/generations/tasks",
"/contents/generations/tasks/task-123",
"/v3/contents/generations/tasks",
"/v3/contents/generations/tasks/task-123",
} {
require.True(t, shouldBypassEmbeddedFrontend(path), "path=%s", path)
}
for _, path := range []string{
"/model-plaza",
"/custom/page-1",
"/monitor",
"/setup",
"/v3/other",
} {
require.False(t, shouldBypassEmbeddedFrontend(path), "path=%s", path)
}
}
func TestNewFrontendServer(t *testing.T) {
t.Run("creates_server_successfully", func(t *testing.T) {
provider := &mockSettingsProvider{
@@ -792,6 +829,9 @@ func TestServeEmbeddedFrontend(t *testing.T) {
"/health",
"/responses",
"/responses/compact",
"/chat/completions",
"/models/gpt-5.5",
"/v3/contents/generations/tasks/task-123",
}
for _, path := range apiPaths {