Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support implicit describe operations #218

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

tvainika
Copy link
Contributor

This implements also implied allow describe + allow describe configs feature as defined by org.apache.kafka.common.acl.AclOperation class.

This mode works only if operations are defined in new mode as list of strings instead of regular expression.

Some of the existing tests have been changed format to ensure backward compatibility by having tests using both formats.

To support all current Kafka operations, Kafka dependency is bumped to 3.6.

This implements also implied allow describe + allow describe configs
feature as defined by org.apache.kafka.common.acl.AclOperation class.

This mode works only if operations are defined in new mode as list of
strings instead of regular expression.

Some of the existing tests have been changed format to ensure backward
compatibility by having tests using both formats.

To support all current Kafka operations, Kafka dependency is bumped to
3.6.
@tvainika tvainika requested review from a team as code owners November 13, 2024 08:58
Copy link
Contributor

@biggusdonzus biggusdonzus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@biggusdonzus biggusdonzus merged commit bdd0ab4 into main Nov 13, 2024
4 checks passed
@biggusdonzus biggusdonzus deleted the implicit-describe-operations branch November 13, 2024 10:50
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.

2 participants