exceptionfactory commented on PR #9869: URL: https://github.com/apache/nifi/pull/9869#issuecomment-2797408560
Thanks for the thoughtful evaluation of options @philci52. I think there are two different elements to consider. The first aspect is bootstrap.conf. Property values using `java.arg` contain user-controller values, and it seems acceptable to depend the user to introduce quotes when necessary, such as the following: ``` java.arg.100=-Dsystem.property.name=escaped\\ string ``` The second aspect is the standard arguments to the command, which include the following from `StandardProcessBuilderProvider`: 1. Class Path argument containing file paths 2. Log Directory system property containing log directory path 3. Application Properties system property containing file path 4. Management Server Address 5. Additional arguments from bootstrap.conf 6. Application class name to run Arguments 1, 2, and 3 contain paths that may need to be quoted, so for those cases, quoting the value of the argument seems like the way to go. Arguments 4 and 6 do not need to be quoted. The additional arguments in 5 can be addressed as described above. With that background, it seems like making adjustments to `StandardProcessBuilderProvider` could provide a workable solution that should apply to all platforms. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
