Skip to content
This repository was archived by the owner on Aug 23, 2023. It is now read-only.

Kafka sasl support#1956

Merged
Dieterbe merged 22 commits intografana:masterfrom
BouyguesTelecom:kafka-sasl-support
Jan 6, 2021
Merged

Kafka sasl support#1956
Dieterbe merged 22 commits intografana:masterfrom
BouyguesTelecom:kafka-sasl-support

Conversation

@Antoine-Auffret
Copy link
Copy Markdown
Contributor

Add support for kafka input SASL mechanism SCRAM-SHA-256 and SCRAM-SHA-512

@Antoine-Auffret
Copy link
Copy Markdown
Contributor Author

Antoine-Auffret commented Jan 5, 2021

Fix breaking changes in ineffassign : gordonklaus/ineffassign#51
Without this fix, the qa job will fail. This must be patched ASAP. @Dieterbe

@Dieterbe
Copy link
Copy Markdown
Contributor

Dieterbe commented Jan 5, 2021

this looks really good.
can you add validation for ' saslMechanism' to abort if it's not set properly? (e.g. no mechanism or an unrecognized mechanism)

@Antoine-Auffret
Copy link
Copy Markdown
Contributor Author

this looks really good.
can you add validation for ' saslMechanism' to abort if it's not set properly? (e.g. no mechanism or an unrecognized mechanism)

Thanks for your feedback, I will make those changes

@Dieterbe Dieterbe merged commit 348e8b5 into grafana:master Jan 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants