Skip to content

feat: add support for security schemes in Kafka config#158

Merged
derberg merged 3 commits intoasyncapi:masterfrom
dalelane:master
Oct 28, 2021
Merged

feat: add support for security schemes in Kafka config#158
derberg merged 3 commits intoasyncapi:masterfrom
dalelane:master

Conversation

@dalelane
Copy link
Contributor

@dalelane dalelane commented Sep 2, 2021

This commit adds support for the new security schemes introduced
in AsyncAPI 2.1.0, for code generated for Kafka servers.

When provided with AsyncAPI specification documents that describe
Kafka clusters requiring authentication, the generator will now
be able to generate the correct configuration for client code.

I'm not familiar with Spring, so I'm absolutely open to suggestions
about whether the structure for these changes is appropriate.

Signed-off-by: Dale Lane [email protected]

This commit adds support for the new security schemes introduced
in AsyncAPI 2.1.0, for code generated for Kafka servers.

When provided with AsyncAPI specification documents that describe
Kafka clusters requiring authentication, the generator will now
be able to generate the correct configuration for client code.

Signed-off-by: Dale Lane <[email protected]>
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@derberg
Copy link
Member

derberg commented Sep 6, 2021

@Tenischev hey man, have time for a review?

@dalelane thanks! let's wait for @Tenischev and if there is no response in few days we will just approve and merge this PR. I know that @Tenischev is not round for some time already but let us make sure for the last time

@derberg derberg merged commit ce7fc91 into asyncapi:master Oct 28, 2021
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 0.24.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants