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

Unify userProperties and tls-version separation #75

Closed
gitseti opened this issue Sep 10, 2019 · 0 comments
Closed

Unify userProperties and tls-version separation #75

gitseti opened this issue Sep 10, 2019 · 0 comments
Assignees
Labels
feature New feature

Comments

@gitseti
Copy link
Contributor

gitseti commented Sep 10, 2019

Problem or use case

  1. Currently, if more than one user property must be specified they must be separated with a "|" (pipe) symbol.

  2. If more than one tls-version to support must be specified it must be separated with a ":" (colon) symbol.

The different separators may result in a confused user and is not an expected behavior.

Preferred solution or suggestions

Use the same separator for both options or omit the separation principle completely and instead require the user to specify the option more than once,
e. g:

hivemq-cli pub -t topic -m msg -up key1=value1 -up key2=value2 -up key3=value3
@gitseti gitseti added feature New feature good first issue Good for newcomers labels Sep 10, 2019
@ahelmbr ahelmbr removed the good first issue Good for newcomers label Sep 18, 2019
@ahelmbr ahelmbr added this to To do in Version 1.0.0 via automation Sep 18, 2019
@gitseti gitseti moved this from To do to In progress in Version 1.0.0 Sep 19, 2019
@gitseti gitseti self-assigned this Sep 19, 2019
@gitseti gitseti moved this from In progress to In Review in Version 1.0.0 Sep 19, 2019
@gitseti gitseti closed this as completed Sep 23, 2019
@gitseti gitseti moved this from In Review to Done in Version 1.0.0 Sep 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
No open projects
Version 1.0.0
  
Version 1.0.0
Development

No branches or pull requests

2 participants