Skip to content

Evaluate nimbus.groups when nimbus.users is empty - #8998

Open
rzo1 wants to merge 1 commit into
masterfrom
fix/nimbus-acl-groups-without-users
Open

Evaluate nimbus.groups when nimbus.users is empty#8998
rzo1 wants to merge 1 commit into
masterfrom
fix/nimbus-acl-groups-without-users

Conversation

@rzo1

@rzo1 rzo1 commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

SimpleACLAuthorizer.permit() short-circuited on nimbusUsers.size() == 0 and allowed the call, so nimbus.groups was never consulted when nimbus.users was left empty. An operator locking the cluster down by group alone therefore got no restriction at all.

All four combinations are now evaluated; both-empty still means no restriction, which is the shipped default. Extends SimpleACLAuthorizerTest.

Note: a cluster that set nimbus.groups without nimbus.users will now restrict user-level commands to members of those groups. That includes NimbusClient.getConfiguredClientAs, which calls getLeader() on every connection.

@rzo1 rzo1 added this to the 3.1.0 milestone Aug 22, 2026
@rzo1 rzo1 self-assigned this Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant