NIFI-145: Make nifi.pid writable by owner

Project: http://git-wip-us.apache.org/repos/asf/incubator-nifi/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-nifi/commit/c1681d95
Tree: http://git-wip-us.apache.org/repos/asf/incubator-nifi/tree/c1681d95
Diff: http://git-wip-us.apache.org/repos/asf/incubator-nifi/diff/c1681d95

Branch: refs/heads/nifi-27
Commit: c1681d95f690a971e9a54e424e77661124077c9d
Parents: 1a231d8
Author: Mark Payne <[email protected]>
Authored: Tue Dec 16 14:36:38 2014 -0500
Committer: Mark Payne <[email protected]>
Committed: Tue Dec 16 14:36:38 2014 -0500

----------------------------------------------------------------------
 .../src/main/java/org/apache/nifi/bootstrap/RunNiFi.java      | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-nifi/blob/c1681d95/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java
----------------------------------------------------------------------
diff --git 
a/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java 
b/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java
index 270ba48..437493e 100644
--- a/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java
+++ b/nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java
@@ -36,9 +36,11 @@ import java.nio.file.attribute.PosixFilePermission;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Properties;
+import java.util.Set;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.locks.Condition;
 import java.util.concurrent.locks.Lock;
@@ -217,7 +219,10 @@ public class RunNiFi {
            }
 
            try {
-               Files.setPosixFilePermissions(statusFile.toPath(), 
Collections.singleton(PosixFilePermission.OWNER_READ));
+               final Set<PosixFilePermission> perms = new HashSet<>();
+               perms.add(PosixFilePermission.OWNER_READ);
+               perms.add(PosixFilePermission.OWNER_WRITE);
+               Files.setPosixFilePermissions(statusFile.toPath(), perms);
            } catch (final Exception e) {
                logger.warning("Failed to set permissions so that only the 
owner can read status file " + statusFile + "; this may allows others to have 
access to the key needed to communicate with NiFi. Permissions should be 
changed so that only the owner can read this file");
            }

Reply via email to