Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2201#discussion_r163569718
--- Diff:
nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
---
@@ -461,8 +463,13 @@ public File getKerberosConfigurationFile() {
// if there was existing validation errors and the processor
loaded successfully
if (currentValidationResults.isEmpty() && instance != null) {
try {
- // defer to the underlying processor for validation
- final Collection<ValidationResult> instanceResults =
instance.validate(context);
+ // defer to the underlying processor for validation,
without the
+ // invokescriptedprocessor properties
+ final Set<PropertyDescriptor> innerPropertyDescriptor =
new HashSet<PropertyDescriptor>(scriptingComponentHelper.getDescriptors());
+
+ ValidationContext innerValidationContext = new
FilteredPropertiesValidationContextAdapter(context, innerPropertyDescriptor);
+ final Collection<ValidationResult> instanceResults =
instance.validate(innerValidationContext);
--- End diff --
I think we should do the same in the `onTrigger` method too
---
