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

[v22.1.x] kafka: validate user provided create_topic configuration literals #5659

Merged

Conversation

vbotbuildovich
Copy link
Collaborator

Backport from pull request: #4953.
Fixes #5658,

Vlad Lazar added 2 commits July 26, 2022 17:46
Some of the topic creation configurations take string literals as
values. Prior to this patch creating a topic with an unexpected string
literal value failed without returning any error message due to an
unhandled exception thrown in the parsing of the literal.

This patch introduces validators which ensure that the string literal
is a valid configuration value and return a meaningful error otherwise.

Fixes redpanda-data#3081

(cherry picked from commit 0343702)
Add tests for topic creation requests that use unexpected literal
values (e.g. `compression.type=foo`).

(cherry picked from commit bac883a)
@vbotbuildovich vbotbuildovich added this to the v22.1.x-next milestone Jul 26, 2022
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Jul 26, 2022
@VladLazar VladLazar marked this pull request as ready for review July 26, 2022 17:46
@VladLazar VladLazar requested a review from dotnwat as a code owner July 26, 2022 17:46
Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

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

This is a backport

@BenPope
Copy link
Member

BenPope commented Jul 26, 2022

Failure on release-clang-amd64 is: #5654
Failure on debug-clang-amd64 appears to be due to a change in the base image
Not waiting for the arm build, as I want to rebuild #5639 today

@BenPope BenPope merged commit 2429eb4 into redpanda-data:v22.1.x Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants