Github user MikeThomsen commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2705#discussion_r188393536
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AbstractJsonPathProcessor.java
---
@@ -112,10 +111,10 @@ static String getResultRepresentation(Object
jsonPathResult, String defaultValue
public ValidationResult validate(final String subject, final
String input, final ValidationContext context) {
String error = null;
if (isStale(subject, input)) {
- if (JsonPathExpressionValidator.isValidExpression(input)) {
+ try {
JsonPath compiledJsonPath = JsonPath.compile(input);
cacheComputedValue(subject, input, compiledJsonPath);
--- End diff --
I should have a `isBlank` check there, so good catch.
---
