This repository was archived by the owner on Jul 22, 2021. It is now read-only.
NIFIREG-75 Fix User Groups for Composite Provider Config#64
Closed
kevdoran wants to merge 1 commit intoapache:masterfrom
Closed
NIFIREG-75 Fix User Groups for Composite Provider Config#64kevdoran wants to merge 1 commit intoapache:masterfrom
kevdoran wants to merge 1 commit intoapache:masterfrom
Conversation
kevdoran
changed the title
Contributor
Author
|
This has been updated with the latest approach |
kevdoran
changed the title
Contributor
|
Reviewing... |
Removes user existence check from FileUserGroupProvider when group is created or updated. Replaces it with check in the Authorizer Decorator class created by Authorizer Factory, so that all providers are used. Also fixes bug when searching for group membership by user that returns results across all providers. Also updates a package in the authorizers.xml template. UI fixes for action icon enabled/disabled states in Administration.
Contributor
|
+1 Ran through a full test with NiFi & Registry both configured with a composite user-group-provider, and put an LDAP user into a file-based group, and then made a bucket and gave the group access to that bucket, verified the LDAP user could use that bucket from NiFi... nice work |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes user existence check from FileUserGroupProvider when group is created or updated. Replaces it with check in the Authorizer Decorator class created by Authorizer Factory, so that all providers are used. Also fixes bug when searching for group membership by user that returns results across all providers.
Also updates a package name in the authorizers.xml template.
UI fix for group membership deletion icon in user side panel.