Skip to content
This repository has been archived by the owner on Mar 29, 2023. It is now read-only.

Kafka auth #510

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Kafka auth #510

wants to merge 19 commits into from

Conversation

DM-sb
Copy link
Collaborator

@DM-sb DM-sb commented Sep 22, 2021

What does this PR change?

Adds support for authentication. At present we only have support for SASL PLAIN mechanism.

Why is it important?

Allows the different kiln components to authenticate to the broker.

Checklist

  • Tests added/updated as appropriate
  • Documentation added/updated as appropriate

If this PR introduces a new tool:

  • Documentation on how to handle false positives added
  • Documentation on how to configure the tool added

@@ -8,7 +8,7 @@ kiln_lib = { git = "https://github.com/simplybusiness/Kiln", features = [ "json"
clap = "2"
chrono = "0.4"
reqwest = { version = "0.11", features = [ "blocking", "json",] }
git2 = "0.13"
git2 = "0.12"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

At present the CI/CD build for data forwarder fails without this change. Potentially related to this rust-lang/rust#85574

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant