refactor parameter check
This commit is contained in:
parent
fdf036d366
commit
edf2cdf755
63
router.go
63
router.go
|
@ -69,59 +69,67 @@ func (s *Server) route(res http.ResponseWriter, req *http.Request) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
/* (4) Check arguments
|
/* (4) Check parameters
|
||||||
---------------------------------------------------------*/
|
---------------------------------------------------------*/
|
||||||
var paramError Err = ErrSuccess
|
var paramError Err = ErrSuccess
|
||||||
|
parameters := make(map[string]interface{})
|
||||||
for name, param := range method.Parameters {
|
for name, param := range method.Parameters {
|
||||||
fmt.Printf("- parameter '%s'\n", name)
|
|
||||||
|
|
||||||
/* (1) Extract value */
|
/* (1) Extract value */
|
||||||
p, isset := request.Data.Set[name]
|
p, isset := request.Data.Set[name]
|
||||||
|
|
||||||
/* (2) OPTIONAL ? */
|
/* (2) Required & missing */
|
||||||
if !isset {
|
if !isset && !*param.Optional {
|
||||||
|
|
||||||
// fail if required
|
|
||||||
if !*param.Optional {
|
|
||||||
fmt.Printf(" - required and missing\n")
|
|
||||||
paramError = ErrMissingParam
|
paramError = ErrMissingParam
|
||||||
paramError.BindArgument(name)
|
paramError.BindArgument(name)
|
||||||
break
|
break
|
||||||
|
}
|
||||||
|
|
||||||
// error if default param is nil
|
/* (3) Optional & missing: set default value */
|
||||||
} else if param.Default == nil {
|
if !isset {
|
||||||
fmt.Printf(" - required and no default\n")
|
|
||||||
paramError = ErrInvalidDefaultParam
|
|
||||||
paramError.BindArgument(name)
|
|
||||||
break
|
|
||||||
|
|
||||||
// set default p if optional
|
|
||||||
} else {
|
|
||||||
fmt.Printf(" - default value: '%v'\n", *param.Default)
|
|
||||||
p = &RequestParameter{
|
p = &RequestParameter{
|
||||||
Parsed: true,
|
Parsed: true,
|
||||||
Value: *param.Default,
|
File: param.Type == "FILE",
|
||||||
|
Value: nil,
|
||||||
|
}
|
||||||
|
if param.Default != nil {
|
||||||
|
p.Value = *param.Default
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
/* (4) Parse parameter if not file */
|
||||||
|
|
||||||
/* (3) Parse parameter variable */
|
|
||||||
if !p.Parsed && !p.File {
|
if !p.Parsed && !p.File {
|
||||||
p.Value = parseHttpData(p.Value)
|
p.Value = parseHttpData(p.Value)
|
||||||
}
|
}
|
||||||
|
|
||||||
/* (4) Check type */
|
/* (4) Fail on unexpected multipart file */
|
||||||
isValid := s.Checker.Run(param.Type, p.Value)
|
waitFile, gotFile := param.Type == "FILE", p.File
|
||||||
fmt.Printf(" - valid: %t\n", isValid == nil)
|
if gotFile && !waitFile || !gotFile && waitFile {
|
||||||
if isValid != nil {
|
paramError = ErrInvalidParam
|
||||||
|
paramError.BindArgument(name)
|
||||||
|
paramError.BindArgument("FILE")
|
||||||
|
break
|
||||||
|
}
|
||||||
|
|
||||||
|
/* (5) Do not check if file */
|
||||||
|
if gotFile {
|
||||||
|
parameters[name] = p.Value
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
|
/* (6) Check type */
|
||||||
|
if s.Checker.Run(param.Type, p.Value) != nil {
|
||||||
|
|
||||||
paramError = ErrInvalidParam
|
paramError = ErrInvalidParam
|
||||||
paramError.BindArgument(name)
|
paramError.BindArgument(name)
|
||||||
paramError.BindArgument(param.Type)
|
paramError.BindArgument(param.Type)
|
||||||
paramError.BindArgument(p.Value)
|
paramError.BindArgument(p.Value)
|
||||||
break
|
break
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
parameters[name] = p.Value
|
||||||
|
|
||||||
}
|
}
|
||||||
fmt.Printf("\n")
|
fmt.Printf("\n")
|
||||||
|
|
||||||
|
@ -135,5 +143,8 @@ func (s *Server) route(res http.ResponseWriter, req *http.Request) {
|
||||||
}
|
}
|
||||||
|
|
||||||
fmt.Printf("OK\nplugin: '%si.so'\n", strings.Join(request.ControllerUri, "/"))
|
fmt.Printf("OK\nplugin: '%si.so'\n", strings.Join(request.ControllerUri, "/"))
|
||||||
|
for name, value := range parameters {
|
||||||
|
fmt.Printf(" $%s = %v\n", name, value)
|
||||||
|
}
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue