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.

Sign up for GitHub

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

Jump to bottom

Expand NewMQTTClient function to support CBOR marshaler and unmarshaler #56

Closed
FelixTing opened this issue Aug 17, 2020 · 2 comments
Closed

Expand NewMQTTClient function to support CBOR marshaler and unmarshaler #56

FelixTing opened this issue Aug 17, 2020 · 2 comments
Labels
invalid This doesn't seem right

Comments

@FelixTing
Copy link
Member

Currently, the NewMQTTClient function uses JSON marshaler and unmarshaler by default. Need to have the option to specify the marshaler/unmarshaler(such as CBOR) according to the MessageBusConfig.
@jpwhitemn jpwhitemn added the hold Intended for PRs we want to flag for ongoing review label Aug 20, 2020
@jpwhitemn
Copy link
Member

Need comes from a customer that needs MQTT trigger

@cloudxxx8 cloudxxx8 added invalid This doesn't seem right and removed hold Intended for PRs we want to flag for ongoing review labels Aug 21, 2020
@cloudxxx8
Copy link
Member

go-mod-messaging is for edgex internal use, so we opened another issue in edgexfoundry/app-functions-sdk-go#454 to fulfill the requirement

this currently is as design

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jpwhitemn @cloudxxx8 @FelixTing