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

Connecting to server URL without port results in CLI error #6319

Closed
mikebin opened this issue Sep 30, 2020 · 2 comments · Fixed by #6410
Closed

Connecting to server URL without port results in CLI error #6319

mikebin opened this issue Sep 30, 2020 · 2 comments · Fixed by #6410
Labels
bug P0 Denotes must-have for a given milestone

Comments

@mikebin
Copy link

mikebin commented Sep 30, 2020

Describe the bug
Connecting to a server URL without a port number (defaulting to 80 or 443) results in an error: Server URL must begin with protocol (e.g., http:// or https://) in ksqlDB 6.0.0.

To Reproduce
Steps to reproduce the behavior, include:
Version: 6.0.0 (CLI and Server)

ksql http://localhost

                  ===========================================
                  =       _              _ ____  ____       =
                  =      | | _____  __ _| |  _ \| __ )      =
                  =      | |/ / __|/ _` | | | | |  _ \      =
                  =      |   <\__ \ (_| | | |_| | |_) |     =
                  =      |_|\_\___/\__, |_|____/|____/      =
                  =                   |_|                   =
                  =  Event Streaming Database purpose-built =
                  =        for stream processing apps       =
                  ===========================================

Copyright 2017-2020 Confluent Inc.

CLI v6.0.0, Server v<unknown> located at http://localhost

Having trouble? Type 'help' (case-insensitive) for a rundown of how things work!

Server URL must begin with protocol (e.g., http:// or https://)
ksql> show topics;
port p must be in range 0 <= p <= 65535

Expected behavior
Successful connection using standard implicit port (80 or 443).

Actual behaviour
See above.

Additional context
Workaround is to add the implicit port number to the URL:

ksql http://localhost:80

This issue does not occur in earlier CLI versions.

@AlanConfluent
Copy link
Member

Is this a regression?

@mikebin
Copy link
Author

mikebin commented Sep 30, 2020

Seems to be, port number is not required in CLI 5.5.1

@apurvam apurvam added P0 Denotes must-have for a given milestone and removed needs-triage labels Sep 30, 2020
ouertani added a commit to ouertani/ksql that referenced this issue Oct 13, 2020
ouertani added a commit to ouertani/ksql that referenced this issue Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug P0 Denotes must-have for a given milestone
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants