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

Shell ignores extra parameters #8593

Closed
aseigneurin opened this issue Jul 12, 2017 · 2 comments
Closed

Shell ignores extra parameters #8593

aseigneurin opened this issue Jul 12, 2017 · 2 comments

Comments

@aseigneurin
Copy link

Feature Request

Proposal: trigger an error when launching the shell with extra parameters (arguments, not options).

Current behavior: when you launch the shell, arguments are simply ignored. This can be misleading.

Each database has its own CLI, and you could forget whether you need to pass the hostname to connect to as an option or as an argument. If you pass the hostname as an argument and have a local InfluxDB database running, it will connect to your local DB, but you might think it successfully connected to the remote host.

$ influx host_I_want_to_connect_to
Connected to http://localhost:8086 version v1.3.0
InfluxDB shell version: v1.3.0
>

Desired behavior: an error would be ideal. It should print the usage and indicate that the extra argument is not valid.

Use case: this just saves time in case people make a mistake in their command line.

@stop-start
Copy link

Hi! I would like to take this if nobody is working on it.

@mark-rushakoff
Copy link
Contributor

@stop-start I believe everyone on the Influx team is focused on other efforts right now - feel free to pick this up if you'd like. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants