markap14 commented on code in PR #7315:
URL: https://github.com/apache/nifi/pull/7315#discussion_r1229690788


##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-components/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java:
##########
@@ -4422,6 +4426,26 @@ public QueueSize getQueueSize() {
         return new QueueSize(count, contentSize);
     }
 
+    @Override
+    public Boolean isLogToOwnFile() {
+        return logToOwnFile;
+    }
+
+    @Override
+    public void setLogToOwnFile(final Boolean logToOwnFile) {
+        this.logToOwnFile = logToOwnFile;
+    }
+
+    @Override
+    public String getLogFileSuffix() {
+        return logFileSuffix;
+    }
+
+    @Override
+    public void setLogFileSuffix(final String logFileSuffix) {
+        this.logFileSuffix = logFileSuffix;

Review Comment:
   We should disallow any characters that are not filename friendly. For 
example, we should not allow spaces, slashes (`/`) or backslashes (`\`)



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to