hazmat345 commented on code in PR #9566:
URL: https://github.com/apache/nifi/pull/9566#discussion_r1881071991
##########
nifi-registry/nifi-registry-core/nifi-registry-security-api/src/main/java/org/apache/nifi/registry/security/authorization/UserGroupProvider.java:
##########
@@ -75,6 +75,30 @@ public interface UserGroupProvider {
*/
Group getGroup(String identifier) throws AuthorizationAccessException;
+ /**
+ * Retrieves a Group by name.
+ *
+ * @param name the name of the group to retrieve
+ * @return the Group with the given name, or null if no matching group was
found
+ * @throws AuthorizationAccessException if there was an unexpected error
performing the operation
+ */
+ default Group getGroupByName(String name) throws
AuthorizationAccessException {
Review Comment:
Ahh, right. That was a changed I pulled from the nifi major OIDC rewrite but
I agree it doesn't make sense to do here at this point. I've removed this
method from the interface and moved the logic into the
`StandardManagedAuthorizer`.
--
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]
