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

[Bug] MQTTX CLI won't recognize topic -t or --topic when used with -lo for loading connection options #1809

Closed
oising opened this issue Nov 14, 2024 · 2 comments
Assignees
Labels
bug Something isn't working CLI MQTTX CLI
Milestone

Comments

@oising
Copy link

oising commented Nov 14, 2024

What did I do

image

What happened

Get an error.

Expected

mqttx should allow me to pick a topic? lol

Environment

as my other bugs, SL7, node 22, win11 on arm64.

More detail

More detail include log information or screenshot

@oising oising added the bug Something isn't working label Nov 14, 2024
@github-project-automation github-project-automation bot moved this to Backlog in MQTTX Nov 14, 2024
@Red-Asuka Red-Asuka self-assigned this Nov 15, 2024
@Red-Asuka Red-Asuka added the CLI MQTTX CLI label Nov 15, 2024
@ysfscream ysfscream added this to the v1.11.1 milestone Nov 15, 2024
@ysfscream ysfscream moved this from Backlog to TO DO in MQTTX Nov 15, 2024
@Red-Asuka
Copy link
Member

Red-Asuka commented Nov 15, 2024

@oising Thank you for your feedback. Based on the description, I guess you may have manually deleted the topic setting from the mqttx-cli-options.json configuration file. Currently, the logic is completely overridden by the configuration file over command line options, resulting in the system prompting that the topic does not exist. What you pointed out is indeed an issue that needs to be optimized. Ideally, the priority of command line parameters should be higher than the settings in the configuration file. We will fix this issue in the next version. As a temporary solution, you can add the topic setting back to the configuration file.

@ysfscream ysfscream moved this from TO DO to Done in MQTTX Nov 15, 2024
@ysfscream ysfscream moved this from Done to In Progress in MQTTX Nov 15, 2024
@ysfscream
Copy link
Member

It has been optimized https://github.com/emqx/MQTTX/releases/tag/v1.11.1, thanks

@github-project-automation github-project-automation bot moved this from In Progress to Done in MQTTX Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CLI MQTTX CLI
Projects
Status: Done
Development

No branches or pull requests

3 participants