From 2a828e2798d5f77e768e1199d88a00996bd3e45a Mon Sep 17 00:00:00 2001 From: wxiaoguang Date: Tue, 24 Dec 2024 21:47:45 +0800 Subject: 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. --- modules/setting/setting.go | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'modules/setting') diff --git a/modules/setting/setting.go b/modules/setting/setting.go index c93d199b1b..20da796b58 100644 --- a/modules/setting/setting.go +++ b/modules/setting/setting.go @@ -235,3 +235,9 @@ func checkOverlappedPath(name, path string) { } configuredPaths[path] = name } + +func PanicInDevOrTesting(msg string, a ...any) { + if !IsProd || IsInTesting { + panic(fmt.Sprintf(msg, a...)) + } +} -- cgit v1.2.3