diff --git a/internal/route/api/v1/api.go b/internal/route/api/v1/api.go index 79d346fe6..ae9b2969f 100644 --- a/internal/route/api/v1/api.go +++ b/internal/route/api/v1/api.go @@ -148,7 +148,7 @@ func reqRepoWriter() macaron.Handler { } } -// reqRepoWriter makes sure the context user has at least admin access to the repository. +// reqRepoAdmin makes sure the context user has at least admin access to the repository. func reqRepoAdmin() macaron.Handler { return func(c *context.Context) { if !c.Repo.IsAdmin() { @@ -158,6 +158,16 @@ func reqRepoAdmin() macaron.Handler { } } +// reqRepoOwner makes sure the context user has owner access to the repository. +func reqRepoOwner() macaron.Handler { + return func(c *context.Context) { + if !c.Repo.IsOwner() { + c.Status(http.StatusForbidden) + return + } + } +} + func mustEnableIssues(c *context.APIContext) { if !c.Repo.Repository.EnableIssues || c.Repo.Repository.EnableExternalTracker { c.NotFound() @@ -250,7 +260,7 @@ func RegisterRoutes(m *macaron.Macaron) { m.Group("/repos", func() { m.Post("/migrate", bind(form.MigrateRepo{}), repo.Migrate) - m.Delete("/:username/:reponame", repoAssignment(), repo.Delete) + m.Delete("/:username/:reponame", repoAssignment(), reqRepoOwner(), repo.Delete) m.Group("/:username/:reponame", func() { m.Group("/hooks", func() {