Skip to content

fix: remove invalid x- fields from Kafka bindings example#1170

Open
wei123-web wants to merge 1 commit intoasyncapi:masterfrom
wei123-web:fix-kafka-example-x-fields
Open

fix: remove invalid x- fields from Kafka bindings example#1170
wei123-web wants to merge 1 commit intoasyncapi:masterfrom
wei123-web:fix-kafka-example-x-fields

Conversation

@wei123-web
Copy link


title: ""
---fix: remove invalid x- fields from Kafka bindings example


Related issue(s):
The ADEO Kafka example had invalid x- fields in Kafka bindings.
These fields are not allowed according to the Kafka binding JSON schema.
Removed x-key.subject.name.strategy and x-value.subject.name.strategy.

Related issue(s): #1164


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 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.

@sonarqubecloud
Copy link

@wei123-web
Copy link
Author

Hello! AsyncAPI maintainers

This PR fixes the ADEO Kafka example by removing invalid x- fields (x-key.subject.name.strategy and x-value.subject.name.strategy) from the Kafka bindings.
These fields are not allowed according to the Kafka binding JSON schema.

Thanks for reviewing!

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