search

vert-x3 / vertx-kafka-client

Reactive Kafka Client for Vert.x
Apache License 2.0
84 stars 82 forks source link

Fix KafkaClientOptions.toJson() so JsonObject includes config contents #248

Closed lawalib closed 1 year ago

lawalib commented 1 year ago

Motivation:

At present KafkaClientOptions.toJson() always returns an empty JsonObject, even if the config map is not empty. This PR fixes that, so that the returned JsonObject reflects what's in the config map.

As requested by @vietj via https://github.com/vert-x3/vertx-kafka-client/pull/247#issuecomment-1593023227, this PR is a port of PR #247 to the 4.x branch.