[NIFI-9311] Cannot remove scripted properties in InvokeScriptedProcessor

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: Extensions
• Assignee: Mark Payne
• Reporter: Matt Burgess
• Created: 19/Oct/21 15:48
• Updated: 27/Oct/21 21:11
• Resolved: 27/Oct/21 21:09

Description

If scripted properties are removed from a script in InvokeScriptedProcessor, they do not become unsupported properties when the script is reloaded. For example, load the following script:

class GroovyProcessor implements Processor {
def ComponentLog log

def REL_SUCCESS = new Relationship.Builder().name("success")
.description("FlowFiles that were successfully processed").build()

@Override
Set<Relationship> getRelationships()

{ return [REL_SUCCESS] as Set }

def PASS = new PropertyDescriptor.Builder().name('Password')
.required(true).sensitive(true).addValidator(Validator.VALID).build()

def NS_USER = new PropertyDescriptor.Builder().name('NS Username')
.required(true).sensitive(false).addValidator(Validator.VALID).build()

@Override
List<PropertyDescriptor> getPropertyDescriptors()

{ return [NS_USER, PASS] as List }

@Override
PropertyDescriptor getPropertyDescriptor(String name) {
switch(name)

{ case 'Password': return PASS case 'NS Username': return NS_USER default: return null }

}

@Override
void onTrigger(ProcessContext context, ProcessSessionFactory sessionFactory) throws ProcessException {
try

{ def session = sessionFactory.createSession() def flowFile = session.get() if(!flowFile) return def pass = context.getProperty(PASS).getValue() // do something with pass session.transfer(flowFile,REL_SUCCESS) session.commit() }

catch (e)

{ throw new ProcessException(e) }

}

@Override
void initialize(ProcessorInitializationContext context)

{ log = context.getLogger() }

@Override
Collection<ValidationResult> validate(ValidationContext context)

{ return null }

@Override
void onPropertyModified(PropertyDescriptor descriptor, String oldValue, String newValue) { }

@Override
String getIdentifier() { return null }

}

processor = new GroovyProcessor()

 

Then go back into the configuration dialog and remove one of the properties, then click Apply. The removed property should now be an unsupported property (not a dynamic property).

Issue Links

• links to: GitHub Pull Request #5488