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

GH-2750: EmbeddedKafka Set Boot's Bootstrap Prop. #2751

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

garyrussell
Copy link
Contributor

Resolves #2750

Previously, setting the bootstrapServersProperty on @EmbeddedKafka (or brokerListProperty on the broker itself) caused the broker to set that property instead of the default property spring.embedded.kafka.brokers.

Now, the custom property is set in addition to the default.

Also, the default property name is spring.kafka.bootstrap-servers. This allows easier testing of Spring Boot applications, without the need for additional configuration.

Resolves spring-projects#2750

Previously, setting the `bootstrapServersProperty` on `@EmbeddedKafka`
(or `brokerListProperty` on the broker itself) caused the broker to
set that property instead of the default property `spring.embedded.kafka.brokers`.

Now, the custom property is set in addition to the default.

Also, the default property name is `spring.kafka.bootstrap-servers`.
This allows easier testing of Spring Boot applications, without the
need for additional configuration.
@artembilan artembilan merged commit 38abddf into spring-projects:main Jul 24, 2023
1 check passed
@garyrussell garyrussell deleted the GH-2750 branch July 24, 2023 19:29
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.

Consider to expose a spring.kafka.bootstrap-servers System property with EmbeddedKafkaBroker
2 participants