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

fix(cli): fix options handler of file write #1639

Merged
merged 1 commit into from
Apr 28, 2024
Merged

Conversation

DM1-1
Copy link
Contributor

@DM1-1 DM1-1 commented Apr 28, 2024

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

The CLI incorrectly handles the --file-write option make subscribe error.

Issue Number

NONE

What is the new behavior?

The CLI tool correctly handles the --file-write option, ensuring that received messages are properly appended to the specified file.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Other information

@DM1-1
Copy link
Contributor Author

DM1-1 commented Apr 28, 2024

image

@DM1-1 DM1-1 marked this pull request as ready for review April 28, 2024 02:33
@DM1-1 DM1-1 requested a review from ysfscream April 28, 2024 02:34
@ysfscream ysfscream requested a review from Red-Asuka April 28, 2024 02:48
@ysfscream ysfscream added fix Fix bug or issues CLI MQTTX CLI labels Apr 28, 2024
@ysfscream ysfscream added this to the v1.10.0 milestone Apr 28, 2024
@ysfscream ysfscream merged commit d60d4b4 into emqx:main Apr 28, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI MQTTX CLI fix Fix bug or issues
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants