You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2020/05/29 08:06:11 UTC

[GitHub] [nifi] simonbence commented on a change in pull request #4303: NIFI-7312: Enable search in variable registry of root process group

simonbence commented on a change in pull request #4303:
URL: https://github.com/apache/nifi/pull/4303#discussion_r432322027



##########
File path: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/java/org/apache/nifi/web/controller/ControllerSearchServiceTest.java
##########
@@ -191,6 +185,7 @@ public void testSearchWhenGroupIsNotAuthorized() {
         testSubject.search(searchQuery, results);
         // The authorization is not transitive, children groups might be good candidates.
         thenFollowingGroupsAreSearched(Arrays.asList(
+                PROCESS_GROUP_ROOT,

Review comment:
       This looks a little strange to my be retrospect, but the documentation says “To reduce the amount of time admins spend on authorization management, policies are inherited from parent resource to child resource”, so I think in practice the case where you cannot access the process group but you can access a resource within should not happen.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org