this is the updateGroup Method , and doUpdateGroup method;
/**
* The group represented by the provided instance will be updated based on
the provided instance.
*
* @param group an updated group instance
* @return the updated group instance, or null if no matching group was
found
* @throws AuthorizationAccessException if there was an unexpected error
performing the operation
* @throws IllegalStateException if there is already a group with the same
name
*/
public final synchronized Group updateGroup(Group group) throws
AuthorizationAccessException {
if (tenantExists(group.getIdentifier(), group.getName())) {
throw new IllegalStateException(String.format("User/user group
already exists with the identity '%s'.", group.getName()));
}
return doUpdateGroup(group);
}
/**
* The group represented by the provided instance will be updated based on
the provided instance.
*
* @param group an updated group instance
* @return the updated group instance, or null if no matching group was
found
* @throws AuthorizationAccessException if there was an unexpected error
performing the operation
*/
public abstract Group doUpdateGroup(Group group) throws
AuthorizationAccessException;
[email protected]
From: Andy LoPresto
Date: 2017-08-02 14:01
To: dev
Subject: Re: a question about nifi's source code
Hi YuNing,
The Abstract policy authorizer delegates the updateGroup action to
doUpdateGroup, which is implemented by the extending concrete class. I'm not
sure where you are seeing that it checks for the absence of the group, but
there is an addGroup method for adding a new group.
Andy LoPresto
[email protected]
[email protected]
PGP Fingerprint: 70EC B3E5 98A6 5A3F D3C4 BACE 3C6E F65B 2F7D EF69
> On Aug 1, 2017, at 19:48, YuNing <[email protected]> wrote:
>
> Hello，everyone
> I am work on reading nifi's source code . it confuse me when i read
> the class "AbstractPolicyBasedAuthorizer", I found there is no method to add
> a user to an existing group. I noticed there is an "updateGroup" method
> ,which i don't think can be used to add users to existing group, because it
> checks the input group to make sure it didn't exist.
> Am i right? can anyone help me, thanks alot !
>
> Best Regards
> YuNing
>
