From 75a772ab4ca63684ec7ca0beb94fb2a913fbfb1d Mon Sep 17 00:00:00 2001 From: mazkev Date: Wed, 28 Oct 2026 08:20:32 +0700 Subject: [PATCH 1/2] test: evaluate symlinks for expected repo path in TestFilesystemStoreOpen --- server/storage_test.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/server/storage_test.go b/server/storage_test.go index 6f48a8d..38eb31b 100644 --- a/server/storage_test.go +++ b/server/storage_test.go @@ -22,8 +22,13 @@ func TestFilesystemStoreOpen(t *testing.T) { if err != nil { t.Fatal(err) } - if repo.Path() != repoPath { - t.Fatalf("expected %q, got %q", repoPath, repo.Path()) + + expectedPath, err := filepath.EvalSymlinks(repoPath) + if err != nil { + t.Fatal(err) + } + if repo.Path() != expectedPath { + t.Fatalf("expected %q, got %q", expectedPath, repo.Path()) } } From e97159c261e20e3614d21008f5a2456c3a4b23cd Mon Sep 17 00:00:00 2001 From: mazkev Date: Wed, 28 Oct 2026 08:33:05 +0700 Subject: [PATCH 2/2] refactor(server): simplify composite literals in services map --- server/server.go | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/server/server.go b/server/server.go index b1d4f52..880dbbf 100644 --- a/server/server.go +++ b/server/server.go @@ -85,17 +85,17 @@ func NewDefault() *Server { } var services = map[string]Service{ - "(.*?)/git-upload-pack$": Service{"POST", serviceRpc, "upload-pack"}, - "(.*?)/git-receive-pack$": Service{"POST", serviceRpc, "receive-pack"}, - "(.*?)/info/refs$": Service{"GET", getInfoRefs, ""}, - "(.*?)/HEAD$": Service{"GET", getTextFile, ""}, - "(.*?)/objects/info/alternates$": Service{"GET", getTextFile, ""}, - "(.*?)/objects/info/http-alternates$": Service{"GET", getTextFile, ""}, - "(.*?)/objects/info/packs$": Service{"GET", getInfoPacks, ""}, - "(.*?)/objects/info/[^/]*$": Service{"GET", getTextFile, ""}, - "(.*?)/objects/[0-9a-f]{2}/[0-9a-f]{38}$": Service{"GET", getLooseObject, ""}, - "(.*?)/objects/pack/pack-[0-9a-f]{40}\\.pack$": Service{"GET", getPackFile, ""}, - "(.*?)/objects/pack/pack-[0-9a-f]{40}\\.idx$": Service{"GET", getIdxFile, ""}, + "(.*?)/git-upload-pack$": {"POST", serviceRpc, "upload-pack"}, + "(.*?)/git-receive-pack$": {"POST", serviceRpc, "receive-pack"}, + "(.*?)/info/refs$": {"GET", getInfoRefs, ""}, + "(.*?)/HEAD$": {"GET", getTextFile, ""}, + "(.*?)/objects/info/alternates$": {"GET", getTextFile, ""}, + "(.*?)/objects/info/http-alternates$": {"GET", getTextFile, ""}, + "(.*?)/objects/info/packs$": {"GET", getInfoPacks, ""}, + "(.*?)/objects/info/[^/]*$": {"GET", getTextFile, ""}, + "(.*?)/objects/[0-9a-f]{2}/[0-9a-f]{38}$": {"GET", getLooseObject, ""}, + "(.*?)/objects/pack/pack-[0-9a-f]{40}\\.pack$": {"GET", getPackFile, ""}, + "(.*?)/objects/pack/pack-[0-9a-f]{40}\\.idx$": {"GET", getIdxFile, ""}, } func Handler() http.HandlerFunc {