From 268c692efde428fa2a00aff0860b093b2f4dc59e Mon Sep 17 00:00:00 2001 From: Bharat Nallan Date: Fri, 13 Mar 2020 07:37:42 -0700 Subject: api: group "/contents" with "" and "/*" #5985 (#5986) This PR groups the contents api endpoint with both `""` and `/*` as valid URL patterns. When `""` is the URL pattern, this means that no repo path has'nt been provided, in which case the path would be the default repo path. When `"/*"` is the URL pattern, then this would return the contents at the specified path if it's valid. Github API v3 is conformant to this behavior and have verified it locally. --- internal/route/api/v1/api.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/internal/route/api/v1/api.go b/internal/route/api/v1/api.go index b54a6dff..98394049 100644 --- a/internal/route/api/v1/api.go +++ b/internal/route/api/v1/api.go @@ -271,7 +271,10 @@ func RegisterRoutes(m *macaron.Macaron) { }, reqRepoAdmin()) m.Get("/raw/*", context.RepoRef(), repo.GetRawFile) - m.Get("/contents/*", repo.GetContents) + m.Group("/contents", func() { + m.Get("", repo.GetContents) + m.Get("/*", repo.GetContents) + }) m.Get("/archive/*", repo.GetArchive) m.Group("/git/trees", func() { m.Get("/:sha", repo.GetRepoGitTree) -- cgit v1.2.3