mosermw commented on code in PR #9868:
URL: https://github.com/apache/nifi/pull/9868#discussion_r2205738516
##########
nifi-framework-bundle/nifi-framework/nifi-file-authorizer/src/main/java/org/apache/nifi/authorization/FileUserGroupProvider.java:
##########
@@ -670,6 +684,7 @@ private synchronized void load() throws JAXBException,
IllegalStateException {
if (emptyTenants) {
populateInitialUsers(tenants);
+ populateInitialGroups(tenants);
Review Comment:
Thanks for considering this @EndzeitBegins. I remember being unsure if there
was an obviously correct approach to this. Looking back now, I agree that
adding all initial users to all initial groups is going too far.
Adding the `Initial Admin Identity` user to the `Initial Admin Group` does
feel like an obvious thing to do, though. If you can make that happen, I think
it makes a lot of sense. If you make this change, consider mentioning it in
the documentation.
--
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]
