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

support to read bootstrap server property when it was defined as a list #376

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

emmanuelsilva
Copy link
Contributor

Resolves #374

Introduces support for reading the boot servers when the property was defined as a List and not a String.

It will avoid a class cast exception when using reactor-kafka with Spring Boot Kafka since the Spring library configures the bootstrap servers as a list of strings and not a String object, as you can see on their implementation.

@pivotal-cla
Copy link

@emmanuelsilva Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@emmanuelsilva Thank you for signing the Contributor License Agreement!

@emmanuelsilva emmanuelsilva changed the title support read bootstrap server config when it is defined as list support to read bootstrap server property when it was defined as a list Dec 20, 2023
Copy link
Contributor

@artembilan artembilan left a comment

Choose a reason for hiding this comment

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

Please, update copyright of the affected classes to the current 2024 and I'll merge this.

Thanks

@artembilan artembilan merged commit 9b09225 into reactor:main Jan 10, 2024
2 checks passed
@artembilan
Copy link
Contributor

@emmanuelsilva ,

thank you for contribution; looking forward for more!

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.

ClassCastException on attempt to get bootstrap server from SenderOption/ReceiverOptions
3 participants