Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

iceCandidatePoolSize: 10 makes no sense #7

Open
fippo opened this issue Mar 25, 2020 · 0 comments
Open

iceCandidatePoolSize: 10 makes no sense #7

fippo opened this issue Mar 25, 2020 · 0 comments

Comments

@fippo
Copy link

fippo commented Mar 25, 2020

https://github.com/webrtc/FirebaseRTC/blob/master/public/app.js#L13
iceCandidatePoolSize: 10,
is pretty much useless and even harmful. Remember that iceCandidatePoolSize is specifying the number of components to pre-gather so a setting of more than one usually does not make sense.

@alvestrand it would be great if you could review whether this code lab is following/promoting/teaching what is to be considered the (unwritten) BCP. See also #1

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

1 participant