[ 
https://issues.apache.org/jira/browse/NIFI-11524?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Nandor Soma Abonyi updated NIFI-11524:
--------------------------------------
    Description: 
This PR indends to add documentation for the ExecuteSteamCommand processor and 
improves the configuration experience by adding dependency between properties.

I've noticed that the processor adds every Dynamic Property as environment 
variable even when it is a command argument. I didn't change this behavior 
because it might introduce a breaking change, but I'd change this behavior in a 
separate 2.0 only PR. Simply adding another regex check there would introduce 
the 4. regex check which is quite ugly so it would be a good opportunity for 
refactor. Another option would be to add validation for environment variables 
to ensure they don't contain {{{}.{}}}character, because most shells don't 
support it. But I'm not sure about that because there are commands that can 
utilize such env variables even though they are not common.

> Documentation and config experience improvement for ExecuteStreamCommand
> ------------------------------------------------------------------------
>
>                 Key: NIFI-11524
>                 URL: https://issues.apache.org/jira/browse/NIFI-11524
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Extensions
>            Reporter: Nandor Soma Abonyi
>            Assignee: Nandor Soma Abonyi
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> This PR indends to add documentation for the ExecuteSteamCommand processor 
> and improves the configuration experience by adding dependency between 
> properties.
> I've noticed that the processor adds every Dynamic Property as environment 
> variable even when it is a command argument. I didn't change this behavior 
> because it might introduce a breaking change, but I'd change this behavior in 
> a separate 2.0 only PR. Simply adding another regex check there would 
> introduce the 4. regex check which is quite ugly so it would be a good 
> opportunity for refactor. Another option would be to add validation for 
> environment variables to ensure they don't contain {{{}.{}}}character, 
> because most shells don't support it. But I'm not sure about that because 
> there are commands that can utilize such env variables even though they are 
> not common.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to