Skip to content

Add and link circuit relay spec#6

Merged
daviddias merged 5 commits intomasterfrom
feat/relay
Feb 13, 2017
Merged

Add and link circuit relay spec#6
daviddias merged 5 commits intomasterfrom
feat/relay

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Feb 8, 2017

Review of the spec itself concluded in libp2p/go-libp2p-circuit#1 -- the additional changes in this PR are purely linking it into place.

I chose to categorize the relay primarily as a transport, since that is what it looks like to the layers using it -- you use it by means of /p2p-circuit multiaddrs, just like other transports. The fact that it's also a swarm protocol is an implementation detail from this point of view.

@ghost ghost added the needs review label Feb 8, 2017
@ghost ghost requested a review from daviddias February 8, 2017 03:24
@ghost ghost added the in progress label Feb 8, 2017

Connection relaying SHOULD be implemented as a transport, in order to be transparent to upper layers.

For an instantiation of relaying, see the [/ipfs/relay/circuit transport](transports/circuit-relay.md).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ipfs/ or /libp2p/?


### 4.2.6 Relay

See [/ipfs/relay/circuit transport](transports/circuit-relay.md).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above?

7-properties.md Outdated
multiplex | /mplex/6.7.0 |
identify | /ipfs/id/1.0.0 |
ping | /ipfs/ping/1.0.0 |
relay | /ipfs/relay/line/0.1.0 | discontinued
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just delete it, as it 'never actually started' :)

@daviddias
Copy link
Copy Markdown
Member

Still a great fan of the dramatization :) I would say that we should take this opportunity to make sure that the protocol strings are very clear and well scoped (e.g relay is a libp2p thing and not an ipfs one)

@daviddias
Copy link
Copy Markdown
Member

@lgierth I've updated the links and the multicodec in the multicodecs table. I understand that his spec is still a WIP, but let's make it a WIP with a placeholder in master, so that we have an entry point for it and then update the spec as the implemention develops. (Trying to avoid having 50 spec fragments everywhere)

@daviddias daviddias merged commit 2bfb69c into master Feb 13, 2017
@daviddias daviddias deleted the feat/relay branch February 13, 2017 16:06
@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 13, 2017

Yes totally agree that we should merge this. While the spec is WIP and will be for a while longer, it's definitely good enough to be merged as a first draft.

jxs pushed a commit to jxs/specs that referenced this pull request Aug 8, 2025
jxs pushed a commit to jxs/specs that referenced this pull request Aug 8, 2025
jxs pushed a commit to jxs/specs that referenced this pull request Aug 8, 2025
jxs pushed a commit to jxs/specs that referenced this pull request Aug 8, 2025
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.

1 participant