following up on #544: do not pass through on unhandled case (#550)

getkin · May 30, 2022 · 221a292 · 221a292
1 parent 7f8f768
commit 221a292
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/openapi3filter/req_resp_encoder.go b/openapi3filter/req_resp_encoder.go
@@ -19,9 +19,5 @@ func encodeBody(body interface{}, mediaType string) ([]byte, error) {
 type bodyEncoder func(body interface{}) ([]byte, error)
 
 var bodyEncoders = map[string]bodyEncoder{
-	"application/json": jsonBodyEncoder,
-}
-
-func jsonBodyEncoder(body interface{}) ([]byte, error) {
-	return json.Marshal(body)
+	"application/json": json.Marshal,
 }
diff --git a/openapi3filter/validate_request.go b/openapi3filter/validate_request.go
@@ -148,8 +148,7 @@ func ValidateParameter(ctx context.Context, input *RequestValidationInput, param
 		value = schema.Default
 		req := input.Request
 		switch parameter.In {
-		case openapi3.ParameterInPath:
-			// TODO: no idea how to handle this
+		// case openapi3.ParameterInPath: TODO: no idea how to handle this
 		case openapi3.ParameterInQuery:
 			q := req.URL.Query()
 			q.Add(parameter.Name, fmt.Sprintf("%v", value))