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 pub the wrong base64 formart message #1413

Merged
merged 1 commit into from
Sep 11, 2023
Merged

fix(cli): fix pub the wrong base64 formart message #1413

merged 1 commit into from
Sep 11, 2023

Conversation

ysfscream
Copy link
Member

@ysfscream ysfscream commented Sep 8, 2023

PR Checklist

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

What is the current behavior?

mqttx pub -t testtopic/protobuf -h 127.0.0.1 -m 'A00340168F4' --format hex

image

Issue Number

https://askemq.com/t/topic/5820

What is the new behavior?

image

Please describe the new behavior or provide screenshots.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to review?

Other information

@ysfscream ysfscream added enhancement New feature or request fix Fix bug or issues refactor Refactor code or architecture CLI MQTTX CLI labels Sep 8, 2023
@ysfscream ysfscream added this to the v1.9.6 milestone Sep 8, 2023
@ysfscream ysfscream requested review from Red-Asuka and ni00 September 8, 2023 09:51
@ysfscream ysfscream self-assigned this Sep 8, 2023
@ysfscream ysfscream merged commit 787477a into main Sep 11, 2023
5 checks passed
@ysfscream ysfscream deleted the ysf/dev branch September 11, 2023 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI MQTTX CLI enhancement New feature or request fix Fix bug or issues refactor Refactor code or architecture
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants