Skip to content

Add Kafka TLS/mTLS certificate flags and secured-broker E2E test - #101

Merged
mumoshu merged 3 commits into
mainfrom
enhanced-kafka-auth
Jul 1, 2026
Merged

Add Kafka TLS/mTLS certificate flags and secured-broker E2E test#101
mumoshu merged 3 commits into
mainfrom
enhanced-kafka-auth

Conversation

@mumoshu

@mumoshu mumoshu commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What is the motivation?

Production Kafka clusters often require TLS and mTLS with custom CAs; #70 called out missing certificate CLI support and we still have no integration test against a secured broker after #77 and #89.

What does this change do?

Adds --ssl-* flags for the Kafka source, test harness support for a TLS/SASL broker, and documents TLS usage in docs/kafka.md.

What is your testing strategy?

  • Unit tests for SSL config mapping in kafka-source
  • Integration test test_kafka_sasl_ssl_mtls_sync against SecureKafkaContainer
  • Pre-push cargo clippy on push; CI make test expected to cover the suite

Is this related to any issues?

Closes #70

Have you read the Contributing Guidelines?

mumoshu added 3 commits July 1, 2026 12:21
Closes #70 by wiring ssl.ca/certificate/key CLI options through the consumer
and adding SecureKafkaContainer plus SASL_SSL mTLS integration coverage.
Run the one-off apache/kafka cert container as root so openssl can write
generated keys into the host temp directory mounted at /work.
Use the current uid:gid for the openssl docker run so generated client
keys remain readable to the test process after bind-mount generation.
@mumoshu
mumoshu merged commit d1eaa2f into main Jul 1, 2026
9 checks passed
@mumoshu
mumoshu deleted the enhanced-kafka-auth branch July 1, 2026 23:14
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.

Bug: Sync with Kafka is missing broker auth support

1 participant