Repository: nifi
Updated Branches:
  refs/heads/master d08233b9f -> 7e1ecad73


NIFI-3706: - Removing non-existent resource when converting a legacy authorized 
users file.

This closes #1673.

Signed-off-by: Bryan Bende <[email protected]>


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

Branch: refs/heads/master
Commit: 7e1ecad738396e78b4cf7997548d539cb5df8bf2
Parents: d08233b
Author: Matt Gilman <[email protected]>
Authored: Thu Apr 13 16:14:45 2017 -0400
Committer: Bryan Bende <[email protected]>
Committed: Tue Apr 18 09:46:21 2017 -0400

----------------------------------------------------------------------
 .../java/org/apache/nifi/authorization/RoleAccessPolicy.java     | 1 -
 .../java/org/apache/nifi/authorization/FileAuthorizerTest.java   | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/nifi/blob/7e1ecad7/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/RoleAccessPolicy.java
----------------------------------------------------------------------
diff --git 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/RoleAccessPolicy.java
 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/RoleAccessPolicy.java
index 03186bc..6ee09fe 100644
--- 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/RoleAccessPolicy.java
+++ 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/RoleAccessPolicy.java
@@ -104,7 +104,6 @@ public final class RoleAccessPolicy {
         final Set<RoleAccessPolicy> nifiPolicies = new HashSet<>();
         nifiPolicies.add(new 
RoleAccessPolicy(ResourceType.Controller.getValue(), READ_ACTION));
         nifiPolicies.add(new 
RoleAccessPolicy(ResourceType.SiteToSite.getValue(), READ_ACTION));
-        nifiPolicies.add(new 
RoleAccessPolicy(ResourceType.SiteToSite.getValue(), WRITE_ACTION));
         roleAccessPolicies.put(Role.ROLE_NIFI, 
Collections.unmodifiableSet(nifiPolicies));
 
         return roleAccessPolicies;

http://git-wip-us.apache.org/repos/asf/nifi/blob/7e1ecad7/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/test/java/org/apache/nifi/authorization/FileAuthorizerTest.java
----------------------------------------------------------------------
diff --git 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/test/java/org/apache/nifi/authorization/FileAuthorizerTest.java
 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/test/java/org/apache/nifi/authorization/FileAuthorizerTest.java
index 394aff5..de76d93 100644
--- 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/test/java/org/apache/nifi/authorization/FileAuthorizerTest.java
+++ 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/test/java/org/apache/nifi/authorization/FileAuthorizerTest.java
@@ -355,8 +355,8 @@ public class FileAuthorizerTest {
         assertEquals(3, user6Policies.size());
 
         
assertTrue(user6Policies.containsKey(ResourceType.SiteToSite.getValue()));
-        assertEquals(2, 
user6Policies.get(ResourceType.SiteToSite.getValue()).size());
-        
assertTrue(user6Policies.get(ResourceType.SiteToSite.getValue()).contains(RequestAction.WRITE));
+        assertEquals(1, 
user6Policies.get(ResourceType.SiteToSite.getValue()).size());
+        
assertTrue(user6Policies.get(ResourceType.SiteToSite.getValue()).contains(RequestAction.READ));
 
         final Resource inputPortResource = 
ResourceFactory.getDataTransferResource(
                 ResourceFactory.getComponentResource(ResourceType.InputPort, 
"2f7d1606-b090-4be7-a592-a5b70fb55531", "TCP Input"));

Reply via email to