You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by ma...@apache.org on 2014/12/16 20:36:40 UTC
incubator-nifi git commit: NIFI-145: Make nifi.pid writable by owner
Repository: incubator-nifi
Updated Branches:
refs/heads/bootstrap 1a231d8b9 -> c1681d95f
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/bootstrap
Commit: c1681d95f690a971e9a54e424e77661124077c9d
Parents: 1a231d8
Author: Mark Payne <ma...@hotmail.com>
Authored: Tue Dec 16 14:36:38 2014 -0500
Committer: Mark Payne <ma...@hotmail.com>
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");
}