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

Kafkacloud #28

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

jfbenckhuijsen
Copy link

Motivation

https://issues.jboss.org/browse/AEROGEAR-9867

What

Allows specification of additional properties on @KafkaConfig, so you can specify e.g. security parameters (which are for example needed to connect to Confluent Cloud)

Why

To be able to connect to security protected Kafka clusters/Confluent Cloud

How

Adding an optional array of additional configuration fields to KafkaConfig (@KafkaAdditionalConfig)

Verification Steps

  1. Connect to a security protected Kafka instance without additional settings (fails)
  2. Update with security settings
  3. Connect again (works)

Checklist:

  • [ X ] Code has been tested locally by PR requester
  • [ X ] Changes have been successfully verified by another team member

Progress

  • Finished additional config feature

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.

1 participant