feat(authorization): support authorization of group new consumer protocol group#3448
Merged
robobario merged 5 commits intokroxylicious:mainfrom Mar 23, 2026
Merged
Conversation
Member
Author
|
Keith suggested in call rather than rejecting the regex when non-null, we could cap the RPC version to v0 and reject v1. This is a better solution in the interrim. |
robobario
force-pushed
the
authz-new-consumer-protocol-group
branch
2 times, most recently
from
90c5743 to
58d3349
Compare
Member
Author
|
pushed up capping the ConsumerGroupHeartbeat to v0 rather than blowing up on the subscription regex. I also added:
|
robobario
force-pushed
the
authz-new-consumer-protocol-group
branch
2 times, most recently
from
56cc36d to
873aeea
Compare
Member
Author
|
I'll split out the bugfix into a separate PR to make this smaller edit: #3476 created, I'll rebase onto that |
robobario
force-pushed
the
authz-new-consumer-protocol-group
branch
4 times, most recently
from
a85bbf1 to
57d8c53
Compare
Signed-off-by: Robert Young <robertyoungnz@gmail.com>
Signed-off-by: Robert Young <robertyoungnz@gmail.com>
…scribe When presenting the authorizedOps int (bitset for the group authorized ops), we want to present the most restrictive authorizedOps we can, taking into account the upstream authorizedOps and the authorizer allowed operations. Signed-off-by: Robert Young <robertyoungnz@gmail.com>
Signed-off-by: Robert Young <robertyoungnz@gmail.com>
robobario
force-pushed
the
authz-new-consumer-protocol-group
branch
from
57d8c53 to
b31889f
Compare
Signed-off-by: Robert Young <robertyoungnz@gmail.com>
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Description
Closes #3442
Note: builds on top of #3476
Checklist
Please go through this checklist and make sure all applicable tasks have been done