net/ghttp：Improve the process of parameter parsing

net/ghttp/ghttp.go

@@ -77,11 +77,14 @@ type (
 
 	// handlerFuncInfo contains the HandlerFunc address and its reflection type.
 	handlerFuncInfo struct {
-		Func            HandlerFunc      // Handler function address.
-		Type            reflect.Type     // Reflect type information for current handler, which is used for extensions of the handler feature.
-		Value           reflect.Value    // Reflect value information for current handler, which is used for extensions of the handler feature.
-		IsStrictRoute   bool             // Whether strict route matching is enabled.
-		ReqStructFields []gstructs.Field // Request struct fields.
+		Func              HandlerFunc      // Handler function address.
+		Type              reflect.Type     // Reflect type information for current handler, which is used for extensions of the handler feature.
+		Value             reflect.Value    // Reflect value information for current handler, which is used for extensions of the handler feature.
+		IsStrictRoute     bool             // Whether strict route matching is enabled.
+		ReqStructFields   []gstructs.Field // Request struct fields.
+		defaultTagFields  []gstructs.Field // Language string `d:"go"`
+		inTagFields       []gstructs.Field // Language string `in:"header"`
+		hasValidTagFields bool
 	}
 
 	// HandlerItem is the registered handler for route handling,

net/ghttp/ghttp_request_param.go

@@ -107,6 +107,12 @@ func (r *Request) doParse(pointer interface{}, requestType int) error {
 				return err
 			}
 		}
+		funcInfo := r.serveHandler.Handler.Info
+		if funcInfo.IsStrictRoute {
+			if funcInfo.hasValidTagFields == false {
+				return nil
+			}
+		}
 		// TODO: https://github.com/gogf/gf/pull/2450
 		// Validation.
 		if err = gvalid.New().

net/ghttp/ghttp_request_param_request.go

@@ -192,7 +192,7 @@ func (r *Request) doGetRequestStruct(pointer interface{}, mapping ...map[string]
 
 // mergeDefaultStructValue merges the request parameters with default values from struct tag definition.
 func (r *Request) mergeDefaultStructValue(data map[string]interface{}, pointer interface{}) error {
-	fields := r.serveHandler.Handler.Info.ReqStructFields
+	fields := r.serveHandler.Handler.Info.defaultTagFields
 	if len(fields) > 0 {
 		var (
 			foundKey   string
@@ -240,7 +240,7 @@ func (r *Request) mergeDefaultStructValue(data map[string]interface{}, pointer i
 
 // mergeInTagStructValue merges the request parameters with header or cookie values from struct `in` tag definition.
 func (r *Request) mergeInTagStructValue(data map[string]interface{}, pointer interface{}) error {
-	fields := r.serveHandler.Handler.Info.ReqStructFields
+	fields := r.serveHandler.Handler.Info.inTagFields
 	if len(fields) > 0 {
 		var (
 			foundKey   string

net/ghttp/ghttp_server_service_handler.go

@@ -257,6 +257,7 @@ func (s *Server) checkAndCreateFuncInfo(
 		return funcInfo, err
 	}
 	funcInfo.ReqStructFields = fields
+	parseStructFields(&funcInfo)
 	funcInfo.Func = createRouterFunc(funcInfo)
 	return
 }
@@ -305,6 +306,61 @@ func createRouterFunc(funcInfo handlerFuncInfo) func(r *Request) {
 	}
 }
 
+func parseStructFields(funcInfo *handlerFuncInfo) {
+	reqStructType := funcInfo.Type.In(1)
+	reqStructType = reqStructType.Elem()
+
+	for i := 0; i < len(funcInfo.ReqStructFields); i++ {
+		field := funcInfo.ReqStructFields[i]
+		defaultTag := field.TagDefault()
+		if defaultTag != "" {
+			funcInfo.defaultTagFields = append(funcInfo.defaultTagFields, field)
+		}
+		inTag := field.TagIn()
+		if inTag != "" {
+			funcInfo.inTagFields = append(funcInfo.inTagFields, field)
+		}
+		validTag := field.TagValid()
+		if validTag == "" {
+			// Check for recursively nested structs
+			funcInfo.hasValidTagFields = checkFieldsHasValidTag(field.Field.Type)
+		} else {
+			funcInfo.hasValidTagFields = true
+		}
+	}
+}
+
+func checkFieldsHasValidTag(field reflect.Type) (has bool) {
+	if field.Kind() == reflect.Ptr {
+		field = field.Elem()
+	}
+	switch field.Kind() {
+	// []*struct or map[key]*struct
+	case reflect.Slice, reflect.Map, reflect.Array:
+		field = field.Elem()
+	}
+	if field.Kind() == reflect.Ptr {
+		field = field.Elem()
+	}
+	if field.Kind() != reflect.Struct {
+		return
+	}
+
+	for i := 0; i < field.NumField(); i++ {
+		f := field.Field(i)
+		tag := f.Tag.Get("v")
+		if tag == "" {
+			tag = f.Tag.Get("valid")
+		}
+		if tag != "" {
+			return true
+		}
+		has = checkFieldsHasValidTag(f.Type)
+	}
+
+	return
+}
+
 // trimGeneric removes type definitions string from response type name if generic
 func trimGeneric(structName string) string {
 	var (

net/ghttp/ghttp_z_unit_issue_test.go

@@ -202,7 +202,7 @@ func (r cFoo1) PostTest1(ctx context.Context, req *TemplateCreateReq) (res *Temp
 }
 
 // https://github.com/gogf/gf/issues/1662
-func Test_Issue662(t *testing.T) {
+func Test_Issue1662(t *testing.T) {
 	s := g.Server(guid.S())
 	s.Use(ghttp.MiddlewareHandlerResponse)
 	s.Group("/boot", func(grp *ghttp.RouterGroup) {