Skip to content

Support mqtt protocol#339

Closed
rockey2020 wants to merge 1 commit intomultiformats:mainfrom
rockey2020:master-2
Closed

Support mqtt protocol#339
rockey2020 wants to merge 1 commit intomultiformats:mainfrom
rockey2020:master-2

Conversation

@rockey2020
Copy link

No description provided.

@achingbrain
Copy link
Member

Thanks for opening this. Where did the codec number come from? I can't see a reference to MQTT in the codec table - it would need to be added there first.

@rockey2020
Copy link
Author

I was trying to develop a ws transport component that uses the mqtt protocol, but I got an error saying that mqtt is not supported.

@achingbrain
Copy link
Member

That's because it's not in the codec table - please open a PR to that repo first, once the codec number is agreed upon and merged this one can go ahead.

@achingbrain achingbrain added the status/blocked Unable to be worked further until needs are met label Sep 13, 2023
@achingbrain
Copy link
Member

Now that #425 has been merged, you can experiment with new protocols without needing to PR support in here.

See Adding custom protocols in the README.

I'm going to close this since there's no entry for MQTT in the codec table yet, we can revisit if it gets added.

Refs: multiformats/multiaddr#106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/blocked Unable to be worked further until needs are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants