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

Clients must be identified with all key attributes instead of only the identifier #43

Closed
gitseti opened this issue Aug 23, 2019 · 0 comments
Labels
invalid This doesn't seem right

Comments

@gitseti
Copy link
Contributor

gitseti commented Aug 23, 2019

Input:
hmq> con -i myClient -h mybroker.com -p 1884 -v 3
hmq> pub -i myClient -t test -m msg

Expected:
The publish command uses the previously connected client which is uniquely identified with "myClient".

Result:
Publish connects a new client to default settings.

Reason:
The client lookup in the client cache uses a key consisting of mqtt-version, host, port and identifier instead of using solely the identifier.

@gitseti gitseti added bug Something isn't working invalid This doesn't seem right and removed bug Something isn't working labels Aug 23, 2019
@gitseti gitseti added this to To do in Version 1.0.0 Aug 23, 2019
@gitseti gitseti closed this as completed Aug 26, 2019
@gitseti gitseti moved this from To do to Done in Version 1.0.0 Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
No open projects
Version 1.0.0
  
Version 1.0.0
Development

No branches or pull requests

1 participant