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

Argument parsing issue for nngproxy #1770

Closed
pmf opened this issue May 2, 2024 · 3 comments
Closed

Argument parsing issue for nngproxy #1770

pmf opened this issue May 2, 2024 · 3 comments
Labels
Investigation quiestion remains ambiguous

Comments

@pmf
Copy link

pmf commented May 2, 2024

Describe the bug
The nngproxy subcommand of nanomq_cli does not work for me due to an argument parsing issue. I have to revert the change in e9fe4b1 for nng_proxy.c (i.e. initialize idx as 2 again) to make nanomq_cli nngproxy work.

Expected behavior

(After reverting e9fe4b1 for nng_proxy.c)

# ./nanomq_cli nngproxy pub0 --mqtt_url mqtt-tcp://127.0.0.1:1883 --dial tcp://localhost:3327 -t foo
unable to connect tcp://localhost:3327 Connection refused!
# that's OK, and it works when I have a corresponding listener

Actual Behavior

(Without reverting e9fe4b1 for nng_proxy.c)

# ./nanomq_cli nngproxy pub0 --mqtt_url mqtt-tcp://127.0.0.1:1883 --dial tcp://localhost:3327 -t foo
NNG url is invalid.

To Reproduce
See command lines above.

Environment Details

  • NanoMQ version: 4d750ba
  • Operating system and version: custom embedded Linux (armv7hf) with kernel 5.7.1 (patched)
  • Compiler and language used: n/a
  • testing scenario: n/a

Client SDK
n/a

Additional context
n/a

@JaylinYu JaylinYu added the Investigation quiestion remains ambiguous label May 5, 2024
@JaylinYu
Copy link
Member

JaylinYu commented May 5, 2024

Thanks for identifying issue. will look into it soon

@JaylinYu
Copy link
Member

JaylinYu commented May 9, 2024

issue confirmed, will push a PR. Or you can submit your own PR. plz tell me if you want to take the credit.

JaylinYu added a commit that referenced this issue May 9, 2024
@pmf
Copy link
Author

pmf commented May 9, 2024

issue confirmed, will push a PR.

👍 Thanks

Or you can submit your own PR. plz tell me if you want to take the credit.

Nah.

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

No branches or pull requests

2 participants