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

Adds Textile Thread multiaddr #201

Merged
merged 2 commits into from
Jan 19, 2021

Conversation

carsonfarmer
Copy link
Contributor

@carsonfarmer carsonfarmer commented Oct 28, 2020

The thread protocol is now relatively well-recognized in the community. Perhaps its time to make it official?

The thread protocol is now sufficiently well-recognized enough to warrant inclusion
@Stebalien
Copy link
Member

What is a thread multiaddr used to address? Usually, multiaddrs are used to address endpoints/machines, not resources/content.

However, this issue comes up so frequently I'm writing a proposal to see if we can unify around a "multipath" concept: multiformats/multiformats#55.

@carsonfarmer
Copy link
Contributor Author

Yup, multiformats/multiformats#55 is probably exactly what we're looking for. In theory, a thread multiaddr is used to address a resource available via a particular peer/endpoint. But it does go one step further than "endpoint/machine" to address a resource.

@carsonfarmer
Copy link
Contributor Author

I see the conversation around the above linked "multipath" proposal is somewhat stalled. What is the likelihood of moving forward on that, versus just working with the code proposed in this PR?

@rvagg
Copy link
Member

rvagg commented Jan 13, 2021

@Stebalien you're probably the authority for that question ^, specifically does:

But it does go one step further than "endpoint/machine" to address a resource.

raise thread to being something we can fit in here?

@Stebalien
Copy link
Member

Yes, that's why I made the multipath proposal. It's a simple generalization of multiaddr.

@rvagg rvagg merged commit 909e183 into multiformats:master Jan 19, 2021
@carsonfarmer carsonfarmer deleted the Carson/thread-multiaddr branch January 20, 2021 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants