We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Currently, RP brokers throw an exception of type std::runtime_error when an unexpected SASL authentication failure occurs. https://github.com/vectorizedio/redpanda/blob/bf5953e45f7d59a4755874047e6bc6726d3cde9e/src/v/kafka/server/requests.cc#L199
std::runtime_error
A recent slack conversation discussed that type std::runtime_error reads as if the exception was not handled gracefully.
Instead, use a error type that better reflects the situation for SASL authentication failures in the kafka sub-system.
The text was updated successfully, but these errors were encountered:
NyaliaLui
Successfully merging a pull request may close this issue.
Currently, RP brokers throw an exception of type
std::runtime_error
when an unexpected SASL authentication failure occurs.https://github.com/vectorizedio/redpanda/blob/bf5953e45f7d59a4755874047e6bc6726d3cde9e/src/v/kafka/server/requests.cc#L199
A recent slack conversation discussed that type
std::runtime_error
reads as if the exception was not handled gracefully.Instead, use a error type that better reflects the situation for SASL authentication failures in the kafka sub-system.
The text was updated successfully, but these errors were encountered: