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

[KIP-86] Implements rd_kafka_sasl_set_credentials #4033

Merged
merged 2 commits into from
Nov 9, 2022
Merged

[KIP-86] Implements rd_kafka_sasl_set_credentials #4033

merged 2 commits into from
Nov 9, 2022

Conversation

josvisser66
Copy link
Contributor

@josvisser66 josvisser66 commented Oct 24, 2022

This PR adds a new API call rd_kafka_sasl_set_credentials which can be used to set or reset the credentials used for SASL PLAIN or SCRAM in a Kafka client. This is useful for environments that want to change their credentials (e.g. because the old credentials are going to expire) without destroying the Kafka client. The new credentials will be used the next time the Kafka client needs to make a new connection to a broker. Setting these credentials using the new API call will not break any connections that were set up using the old credentials.

Note: Although this is not the same as KIP-86, it allows for the same sort of behavior as KIP-86.

Copy link
Contributor

@edenhill edenhill left a comment

Choose a reason for hiding this comment

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

Great stuff!

src/rdkafka.c Outdated Show resolved Hide resolved
.idea/codeStyles/codeStyleConfig.xml Outdated Show resolved Hide resolved
src/rdkafka.c Outdated Show resolved Hide resolved
src/rdkafka.c Outdated Show resolved Hide resolved
src/rdkafka.c Outdated Show resolved Hide resolved
src/rdkafka.c Outdated Show resolved Hide resolved
src/rdkafka_sasl_plain.c Outdated Show resolved Hide resolved
@josvisser66 josvisser66 changed the title [KIP-86] Implements rd_kafka_set_sasl_creds [KIP-86] Implements rd_kafka_sasl_set_credentials Oct 27, 2022
@josvisser66 josvisser66 marked this pull request as ready for review October 27, 2022 15:53
CHANGELOG.md Outdated

librdkafka v1.9.4 is a maintenance release:

* Added rd_kafka_sasl_set_credentials API function to set or reset the SASL PLAIN or SCRAM credentials used for new broker connections.
Copy link
Contributor

Choose a reason for hiding this comment

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

This should go in the v1.9.3 section below - that's the next release.
Add it to the Enhancement chapter
Try to stick below 80 columns

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Co-authored-by: Jos Visser <jvisser@confluent.io>
Co-authored-by: edenhill <magnus@edenhill.se>
@edenhill edenhill merged commit 3b50e1e into confluentinc:master Nov 9, 2022
@edenhill
Copy link
Contributor

edenhill commented Nov 9, 2022

Thanks Jos!

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