aboutsummaryrefslogtreecommitdiffstats
path: root/routers/api/v1/admin/cron.go
diff options
context:
space:
mode:
authorwxiaoguang <wxiaoguang@gmail.com>2024-12-24 21:47:45 +0800
committerGitHub <noreply@github.com>2024-12-24 13:47:45 +0000
commit2a828e2798d5f77e768e1199d88a00996bd3e45a (patch)
tree2073e3384d41675feae64dd38e79168ae25b8a33 /routers/api/v1/admin/cron.go
parentb8b690feb958dffb51eb1d5fc86c16996ba830e1 (diff)
downloadgitea-2a828e2798d5f77e768e1199d88a00996bd3e45a.tar.gz
gitea-2a828e2798d5f77e768e1199d88a00996bd3e45a.zip
Clarify path param naming (#32969)
In history (from some legacy frameworks), both `:name` and `name` are supported as path path name, `:name` is an alias to `name`. To make code consistent, now we should only use `name` but not `:name`. Also added panic check in related functions to make sure the name won't be abused in case some downstreams still use them.
Diffstat (limited to 'routers/api/v1/admin/cron.go')
-rw-r--r--routers/api/v1/admin/cron.go2
1 files changed, 1 insertions, 1 deletions
diff --git a/routers/api/v1/admin/cron.go b/routers/api/v1/admin/cron.go
index fba9d33f25..962e007776 100644
--- a/routers/api/v1/admin/cron.go
+++ b/routers/api/v1/admin/cron.go
@@ -74,7 +74,7 @@ func PostCronTask(ctx *context.APIContext) {
// "$ref": "#/responses/empty"
// "404":
// "$ref": "#/responses/notFound"
- task := cron.GetTask(ctx.PathParam(":task"))
+ task := cron.GetTask(ctx.PathParam("task"))
if task == nil {
ctx.NotFound()
return