exceptionfactory commented on PR #6467: URL: https://github.com/apache/nifi/pull/6467#issuecomment-1487476134
@malthe Returning to this pull request after some time, thanks for your patience. On further review, I'm not sure it is a good fit to incorporate as a standard capability. The current implementation does not support returning user information, so it appears limited to just the `Group` aspect of the `UserGroupProvider` interface. Although it could be combined together with the existing `AzureGraphUserGroupProvider`, that is a key configuration factor that may not be immediately obvious. The existing `AzureGraphUserGroupProvider` covers both users and groups, and although this provider could be helpful in some scenarios, I'm not sure if it is common enough to warrant including as a standard component. It might make more sense to publish it separately to Maven Central and see if it gains interest among community users. What do you think? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
