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

Add /p2p, /dns4, /dns6, /wss, /p2p-circuit #46

Closed
wants to merge 2 commits into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Jan 18, 2017

See multiformats/multiaddr#34

The new /exp-dns is a test balloon for /dns and will be renamed once we're sure it's right.

Also removes protocols.csv so that there's one less source of confusion.

@ghost ghost added kind/enhancement A net-new feature or improvement to an existing feature help wanted Seeking public contribution on this issue status/in-progress In progress needs review and removed help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature labels Jan 18, 2017
@ghost ghost force-pushed the feat/dns-relay-wss branch 2 times, most recently from b8d535f to 00cc31f Compare January 19, 2017 02:49
@ghost ghost force-pushed the feat/dns-relay-wss branch from 00cc31f to ed76135 Compare January 22, 2017 03:53
@ghost ghost changed the title Add /ws, /wss, /exp-dns, /libp2p-relay, /p2p Add /p2p, /dns4, /dns6, /wss, /libp2p-circuit-relay Jan 22, 2017
@ghost
Copy link
Author

ghost commented Jan 22, 2017

Updated this as per multiformats/multiaddr#37

The updated scope of this PR is: /p2p, /dns4, /dns6, /wss, /libp2p-circuit-relay

@ghost
Copy link
Author

ghost commented Jan 22, 2017

@whyrusleeping that travis osx failure is odd:

$ make deps
go get github.com/whyrusleeping/gx
package gx/ipfs/QmYDds3421prZgqKbLpEK7T9Aa2eVdQ7o3YarX1LVLdP2J/go-multihash: unrecognized import path "gx/ipfs/QmYDds3421prZgqKbLpEK7T9Aa2eVdQ7o3YarX1LVLdP2J/go-multihash" (import path does not begin with hostname)
make: *** [gx] Error 1

@ghost
Copy link
Author

ghost commented Jan 26, 2017

Hold off, this needs more

Lars Gierth added 2 commits February 12, 2017 04:08
@ghost ghost mentioned this pull request Feb 12, 2017
@ghost ghost changed the title Add /p2p, /dns4, /dns6, /wss, /libp2p-circuit-relay Add /p2p, /dns4, /dns6, /wss, /p2p-circuit Feb 12, 2017
@ghost
Copy link
Author

ghost commented Feb 12, 2017

Updated: includes /p2p, /p2p-circuit, /dns4, /dns6, /ws, /wss

@coveralls
Copy link

coveralls commented Feb 12, 2017

Coverage Status

Coverage decreased (-0.2%) to 78.0% when pulling 92dbd0d on feat/dns-relay-wss into 5ea81f9 on master.

@ghost ghost requested review from whyrusleeping and Kubuxu February 14, 2017 01:30
@whyrusleeping
Copy link
Member

i really don't like us having to modify this package every time we make a change to multiaddrs. This package is pluggable, see how i do it in https://github.com/libp2p/go-ws-transport/blob/master/websocket.go

@whyrusleeping
Copy link
Member

Also, i think @jbenet wants to keep the protocols.csv file in there, just for the purpose of making this sort of change more deliberate.

@Kubuxu
Copy link
Member

Kubuxu commented Feb 14, 2017

About removing the protocols.csv, multiple times there was a problem because the unified table was not synced with sub-multiformat table.

@whyrusleeping
Copy link
Member

@Kubuxu are you in favor of, or against the table here?

@Kubuxu
Copy link
Member

Kubuxu commented Feb 14, 2017

I think I am against, we might want to improve the table in multicodec to say which multiformat is responsible for that codec but duplicating the table to each repo will be problematic.

@ghost
Copy link
Author

ghost commented Feb 21, 2017

i really don't like us having to modify this package every time we make a change to multiaddrs. This package is pluggable, see how i do it in https://github.com/libp2p/go-ws-transport/blob/master/websocket.go

Yep excellent point, thanks 👍

What do you think of breaking it up like this?

  • /dns4, /dns6: both go-ipfs and go-multiaddr
    • and lazy-remove the one in go-ipfs with the next go-multiaddr update
  • /p2p: go-libp2p
    • also migrate /ipfs there, then lazy-remove the one here in go-multiaddr
  • /wss: go-ws-transport
  • /p2p-circuit: go-libp2p-circuit (the relay impl)

@ghost
Copy link
Author

ghost commented Mar 5, 2017

For /dns4 and /dns6 (and /dnsaddr) I started https://github.com/multiformats/go-multiaddr-dns, which already works and also has a CLI (madns). Needs tests, and obviously integration with go-libp2p/go-ipfs.

@ghost
Copy link
Author

ghost commented Mar 7, 2017

Closing this as all protocols will be handled in other packages.

@ghost ghost closed this Mar 7, 2017
@ghost ghost removed the status/in-progress In progress label Mar 7, 2017
@ghost ghost deleted the feat/dns-relay-wss branch March 7, 2017 05:59
Stebalien added a commit that referenced this pull request May 20, 2020
private networks and utility functions
This pull request was closed.
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.

3 participants