fixes #129 validator error doesnot have the name for the header, path and query parameters (#130)

Author: stevehu

schema-validator/src/main/java/com/networknt/aws/lambda/validator/RequestValidator.java

@@ -12,6 +12,7 @@
 import com.networknt.openapi.NormalisedPath;
 import com.networknt.openapi.OpenApiOperation;
 import com.networknt.openapi.ValidatorConfig;
+import com.networknt.schema.PathType;
 import com.networknt.schema.SchemaValidatorsConfig;
 import com.networknt.status.Status;
 import com.networknt.utility.MapUtil;
@@ -87,9 +88,11 @@ private Status validateRequestBody(String requestBody, final OpenApiOperation op
             }
             return null;
         }
-        SchemaValidatorsConfig config = new SchemaValidatorsConfig();
-        config.setTypeLoose(false);
-        config.setHandleNullableField(validatorConfig.isHandleNullableField());
+        SchemaValidatorsConfig config = SchemaValidatorsConfig.builder()
+                .typeLoose(false)
+                .pathType(PathType.JSON_POINTER)
+                .nullableKeywordEnabled(validatorConfig.isHandleNullableField())
+                .build();
         // the body can be converted to JsonNode here. If not, an error is returned.
         JsonNode requestNode = null;
         requestBody = requestBody.trim();
@@ -146,7 +149,7 @@ private Status validatePathParameters(final APIGatewayProxyRequestEvent requestE
                     LOG.info("Path parameter cannot be decoded, it will be used directly");
                 }
 
-                return schemaValidator.validate(new TextNode(paramValue), Overlay.toJson((SchemaImpl)(parameter.get().getSchema())));
+                return schemaValidator.validate(new TextNode(paramValue), Overlay.toJson((SchemaImpl)(parameter.get().getSchema())), paramName);
             }
         }
         return status;
@@ -181,7 +184,7 @@ private Status validateQueryParameter(final APIGatewayProxyRequestEvent requestE
                 return new Status(VALIDATOR_REQUEST_PARAMETER_QUERY_MISSING, queryParameter.getName(), openApiOperation.getPathString().original());
             }
         } else {
-            return schemaValidator.validate(new TextNode(queryParameterValue), Overlay.toJson((SchemaImpl)queryParameter.getSchema()));
+            return schemaValidator.validate(new TextNode(queryParameterValue), Overlay.toJson((SchemaImpl)queryParameter.getSchema()), queryParameter.getName());
         }
         return null;
     }
@@ -235,7 +238,7 @@ private Status validateHeader(final APIGatewayProxyRequestEvent requestEvent,
                 return new Status(VALIDATOR_REQUEST_PARAMETER_HEADER_MISSING, headerParameter.getName(), openApiOperation.getPathString().original());
             }
         } else {
-            return headerParameter.getSchema() != null ? schemaValidator.validate(new TextNode(headerValue.get()), Overlay.toJson((SchemaImpl)headerParameter.getSchema())) : null;
+            return headerParameter.getSchema() != null ? schemaValidator.validate(new TextNode(headerValue.get()), Overlay.toJson((SchemaImpl)headerParameter.getSchema()), headerParameter.getName()) : null;
         }
         return null;
     }
@@ -251,7 +254,7 @@ private ValidationResult validateDeserializedValues(final APIGatewayProxyRequest
                     if (null==deserializedValue) {
                         validationResult.addSkipped(p);
                     }else {
-                        Status s = schemaValidator.validate(new TextNode(deserializedValue), Overlay.toJson((SchemaImpl)(p.getSchema())));
+                        Status s = schemaValidator.validate(new TextNode(deserializedValue), Overlay.toJson((SchemaImpl)(p.getSchema())), p.getName());
                         validationResult.addStatus(s);
                     }
                 });

schema-validator/src/main/java/com/networknt/aws/lambda/validator/SchemaValidator.java

@@ -63,8 +63,10 @@ public SchemaValidator() {
     public SchemaValidator(final OpenApi3 api) {
         this.api = api;
         this.jsonNode = Overlay.toJson((OpenApi3Impl)api).get("components");
-        this.defaultConfig = new SchemaValidatorsConfig();
-        this.defaultConfig.setTypeLoose(true);
+        this.defaultConfig = SchemaValidatorsConfig.builder()
+                .typeLoose(true)
+                .pathType(PathType.JSON_POINTER)
+                .build();
     }
 
     /**
@@ -93,8 +95,12 @@ public Status validate(final JsonNode value, final JsonNode schema, SchemaValida
         return doValidate(value, schema, config, instanceLocation);
     }
 
-    public Status validate(final JsonNode value, final JsonNode schema) {
-        return doValidate(value, schema, defaultConfig, null);
+    public Status validate(final JsonNode value, final JsonNode schema, String at) {
+        JsonNodePath instanceLocation = new JsonNodePath(defaultConfig.getPathType());
+        if (at != null) {
+            instanceLocation = instanceLocation.append(at);
+        }
+        return validate(value, schema, defaultConfig, instanceLocation);
     }
 
     public Status validate(final JsonNode value, final JsonNode schema, JsonNodePath instanceLocation) {