Skip to content

feat: Allow SASL handshake version to be influenced from config#292

Merged
birdayz merged 1 commit into
birdayz:masterfrom
k-wall:enable-sasl-v1
Dec 27, 2023
Merged

feat: Allow SASL handshake version to be influenced from config#292
birdayz merged 1 commit into
birdayz:masterfrom
k-wall:enable-sasl-v1

Conversation

@k-wall

@k-wall k-wall commented Dec 13, 2023

Copy link
Copy Markdown
Contributor

why: current kaf uses the older SASL handshake v0, where the SASL payload is sent unframed down the wire. v1 uses the dedicated SASL Kafka RPCs. This feature allows the user to opt for the v1. The default remains as v0.

why: current kaf uses the older SASL handshake v0, where the SASL payload
is sent unframed down the wire.  v1 uses the dedicated SASL Kafka RPCs.  This
feature allows the user to opt for the v1.  The default remains as v0.

@birdayz birdayz left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

thanks!

@birdayz birdayz merged commit 2199481 into birdayz:master Dec 27, 2023
@k-wall k-wall deleted the enable-sasl-v1 branch December 27, 2023 23:39
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