Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

Make waku-web browser work more reliably #8

Open
oskarth opened this issue Oct 19, 2020 · 7 comments
Open

Make waku-web browser work more reliably #8

oskarth opened this issue Oct 19, 2020 · 7 comments

Comments

@oskarth
Copy link
Contributor

oskarth commented Oct 19, 2020

Problem

Currently this setup is quite janky with browser. NodeJS and nim-waku generally works. But for browser we often get:

ICE failed, add a TURN server and see about:webrtc for more details errors and similar, where browser can't establish a connection/discover the node js version.

Acceptance criteria

Make browser work more reliably, as well as documented caveats of why things might fail etc.

@oskarth oskarth changed the title Make web part more reliable Make waku-web browser work more reliably Oct 19, 2020
@oskarth
Copy link
Contributor Author

oskarth commented Oct 19, 2020

cc @decanus who might know more about this wrt wrtc etc

@decanus
Copy link

decanus commented Oct 19, 2020

I in-fact do, so when negotating a webrtc connection you need to have a set of TURN servers. This is essentially a relay if a direct connection cannot be established.

@oskarth
Copy link
Contributor Author

oskarth commented Oct 19, 2020

Isn't this what the signaling server is already doing? Just badly

@decanus
Copy link

decanus commented Oct 19, 2020

No, the signaling server is used to establish connections. The TURN server is a proxy if a direct connection cannot be established. See Turn.

@oskarth
Copy link
Contributor Author

oskarth commented Oct 19, 2020

For some reason I thought TURN was provided for by https://github.com/libp2p/js-libp2p-webrtc-star

It is also worth noting that the *-star family is not recommended

@decanus
Copy link

decanus commented Oct 19, 2020

@oskarth I wouldn't assume it is, as it would need to be some node / server that can be accessed. Not the browser.

@oskarth
Copy link
Contributor Author

oskarth commented Oct 19, 2020

I don't mean the browser, I mean the Node JS bootstrap node. https://github.com/vacp2p/waku-web-chat/tree/master/nodejs/bootstrap

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants