go-libp2p icon indicating copy to clipboard operation
go-libp2p copied to clipboard

Make WebRTC non experimental

Open sukunrt opened this issue 2 years ago • 1 comments

Currently WebRTC has been deployed with an experimental tag and has the following caveat: https://github.com/libp2p/go-libp2p/blob/master/p2p/transport/webrtc/transport.go#L4-L7

// While we're fairly confident that the implementation correctly implements the specification,
// we're not making any guarantees regarding its security (especially regarding resource exhaustion attacks).
// Fixes, even for security-related issues, will be conducted in the open.

We should remove aim to experimental tag and make this transport enabled by default.

sukunrt avatar Nov 27 '23 16:11 sukunrt

To make WebRTC a default transport:

  1. We need to be sure that the resource usage is acceptable (see #1917), both in our code and in the underlying Pion WebRTC and SCTP stack.
  2. #2615 needs to be resolved and merged.
  3. Deployment experience that shows actual usage of the WebRTC transport would be nice. Even better if it comes with metrics.

We should consider bundling this with WebRTC private-to-private. It seems like the above points would most likely be satisfied for both WebRTC flavors at the same time.

marten-seemann avatar Dec 07 '23 12:12 marten-seemann

any update on webrtc @sukunrt ?

SgtPooki avatar Jul 25 '24 18:07 SgtPooki

Targeting release next week.

sukunrt avatar Jul 26 '24 08:07 sukunrt