From 0bae158e41a16fe8df037e4f355b226d2bbf60ad Mon Sep 17 00:00:00 2001
From: Alex X <alexey.khit@gmail.com>
Date: Sun, 16 Nov 2025 19:01:06 +0300
Subject: [PATCH] Code refactoring for #1939

---
 .vscode/launch.json          | 19 -------------------
 internal/streams/api.go      |  8 +-------
 internal/streams/api_test.go | 14 --------------
 internal/streams/handlers.go | 16 +++++++++-------
 4 files changed, 10 insertions(+), 47 deletions(-)
 delete mode 100644 .vscode/launch.json

diff --git a/.vscode/launch.json b/.vscode/launch.json
deleted file mode 100644
index 6242075e..00000000
--- a/.vscode/launch.json
+++ /dev/null
@@ -1,19 +0,0 @@
-{
-    // Use IntelliSense to learn about possible attributes.
-    // Hover to view descriptions of existing attributes.
-    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
-    "version": "0.2.0",
-    "configurations": [
-
-        {
-            "name": "Debug go2rtc",
-            "type": "go",
-            "request": "launch",
-            "mode": "auto",
-            "env": {
-                "CGO_ENABLED": "0"
-            },
-            "program": "main.go",
-        }
-    ]
-}
\ No newline at end of file
diff --git a/internal/streams/api.go b/internal/streams/api.go
index bd9e7f7c..0cc537c3 100644
--- a/internal/streams/api.go
+++ b/internal/streams/api.go
@@ -178,11 +178,5 @@ func apiPreload(w http.ResponseWriter, r *http.Request) {
 }
 
 func apiSchemes(w http.ResponseWriter, r *http.Request) {
-	if r.Method != "GET" {
-		http.Error(w, "", http.StatusMethodNotAllowed)
-		return
-	}
-
-	schemes := GetSupportedSchemes()
-	api.ResponseJSON(w, schemes)
+	api.ResponseJSON(w, SupportedSchemes())
 }
diff --git a/internal/streams/api_test.go b/internal/streams/api_test.go
index 414a9380..2cb93d2a 100644
--- a/internal/streams/api_test.go
+++ b/internal/streams/api_test.go
@@ -35,20 +35,6 @@ func TestApiSchemes(t *testing.T) {
 		require.Contains(t, schemes, "rtmp")
 		require.Contains(t, schemes, "http")
 	})
-
-	t.Run("non-GET requests return method not allowed", func(t *testing.T) {
-		methods := []string{"POST", "PUT", "DELETE", "PATCH"}
-		for _, method := range methods {
-			t.Run(method, func(t *testing.T) {
-				req := httptest.NewRequest(method, "/api/schemes", nil)
-				w := httptest.NewRecorder()
-
-				apiSchemes(w, req)
-
-				require.Equal(t, http.StatusMethodNotAllowed, w.Code)
-			})
-		}
-	})
 }
 
 func TestApiSchemesNoDuplicates(t *testing.T) {
diff --git a/internal/streams/handlers.go b/internal/streams/handlers.go
index 91efb975..9433044b 100644
--- a/internal/streams/handlers.go
+++ b/internal/streams/handlers.go
@@ -2,9 +2,7 @@ package streams
 
 import (
 	"errors"
-	"maps"
 	"regexp"
-	"slices"
 	"strings"
 
 	"github.com/AlexxIT/go2rtc/pkg/core"
@@ -18,15 +16,19 @@ func HandleFunc(scheme string, handler Handler) {
 	handlers[scheme] = handler
 }
 
-func GetSupportedSchemes() []string {
-	unique := make(map[string]bool)
+func SupportedSchemes() []string {
+	uniqueKeys := make(map[string]struct{}, len(handlers)+len(redirects))
 	for scheme := range handlers {
-		unique[scheme] = true
+		uniqueKeys[scheme] = struct{}{}
 	}
 	for scheme := range redirects {
-		unique[scheme] = true
+		uniqueKeys[scheme] = struct{}{}
 	}
-	return slices.Collect(maps.Keys(unique))
+	resultKeys := make([]string, 0, len(uniqueKeys))
+	for key := range uniqueKeys {
+		resultKeys = append(resultKeys, key)
+	}
+	return resultKeys
 }
 
 func HasProducer(url string) bool {