add: security protocol in multiaddr

libp2p / docs

Documentation site for the libp2p project.

https://docs.libp2p.io/

Other

60 stars 88 forks source link

add: security protocol in multiaddr #276

Closed salmad3 closed 1 year ago

salmad3 commented 1 year ago

Context

Addressing-based material for the performance optimization of the security protocol being included in a peer's multiaddr.
resolves of #214, part of #212

Latest preview

Please view the latest Fleek preview here.

salmad3 commented 1 year ago

We don't do this today. I don't think we should merge this until/unless we make these changes.

Right, this is why I put the blocked label.

marten-seemann commented 1 year ago

We don't do this today. I don't think we should merge this until/unless we make these changes.

Right, this is why I put the blocked label.

Can we close it instead? I suggest not to write docs pages for features that we don't have yet, unless that feature is under active development and the rollout is imminent (as it was, for example, with WebRTC and Early Muxer Negotiation).

salmad3 commented 1 year ago

We don't do this today. I don't think we should merge this until/unless we make these changes.

Right, this is why I put the blocked label.

Can we close it instead? I suggest not to write docs pages for features that we don't have yet, unless that feature is under active development and the rollout is imminent (as it was, for example, with WebRTC and Early Muxer Negotiation).

I was under the impression that it was active. Sure, we can close this and reopen if necessary.