-
Notifications
You must be signed in to change notification settings - Fork 1k
Description
Describe the bug
In my organization we rely on a lot of path based routing for microservices, ksqldb being one of them. Due to how the URI is constructed, when the port is not provided, it is appended to the end. For example, https://db-server.com/ksql, becomes https://db-server.com/ksql:443 instead of https://db-server.com:443/ksql. When the port is provided this does not occur, https://db-server.com:443/ksql.
Path based ksqldb server are accepted as arguments but not actually supported so I am not sure how much of an issue this is until #6358 is resolved.
To Reproduce
podman run --platform linux/amd64 -it confluentinc/cp-ksqldb-cli https://db-server.com/ksql
Expected behavior
===========================================
= _ _ ____ ____ =
= | | _____ __ _| | _ \| __ ) =
= | |/ / __|/ _` | | | | | _ \ =
= | <\__ \ (_| | | |_| | |_) | =
= |_|\_\___/\__, |_|____/|____/ =
= |_| =
= The Database purpose-built =
= for stream processing apps =
===========================================
Copyright 2017-2022 Confluent Inc.
CLI v7.8.0, Server v<unknown> located at https://db-server.com:443/ksql
Actual behaviour
===========================================
= _ _ ____ ____ =
= | | _____ __ _| | _ \| __ ) =
= | |/ / __|/ _` | | | | | _ \ =
= | <\__ \ (_| | | |_| | |_) | =
= |_|\_\___/\__, |_|____/|____/ =
= |_| =
= The Database purpose-built =
= for stream processing apps =
===========================================
Copyright 2017-2022 Confluent Inc.
CLI v7.8.0, Server v<unknown> located at https://db-server.com/ksql:443
Additional context
Source of the problem https://github.com/confluentinc/ksql/blob/master/ksqldb-rest-client/src/main/java/io/confluent/ksql/rest/client/KsqlRestClient.java#L389 and identified when working on a PR for #6358