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

Consider webrtc<-->webrtc vs webrtc<-->websocket for the browser proxy #8

Closed
keroserene opened this issue Jan 16, 2016 · 3 comments
Closed
Labels

Comments

Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@keroserene
Copy link
Owner

@keroserene keroserene commented Jan 16, 2016

webrtc <-->websocket is the current situation, which works.
webrtc <--> webrtc would take a bit more work.

@arlolra
Copy link
Collaborator

@arlolra arlolra commented Feb 9, 2016

Quoting @Yawning,

What are your plans for actually getting the server side to scale well? Since you're using cgo you will run into Really Interesting behavior wrt OS threads as you try to increase concurrency.

https://lists.torproject.org/pipermail/tor-dev/2016-January/010311.html

@keroserene
Copy link
Owner Author

@keroserene keroserene commented Feb 10, 2016

Yes, that does complicate the potential of using a second WebRTC connection to the relay...

The benefit of WebRTC on both sides should be performance / latency, whereas right now the websocket connection to the relay is most likely bottlenecking the first WebRTC data channel from the client.

But maybe it's actually not, at least significantly. It's quite possible the typical latency of the network is already most of what the user experiences compared to the websocket relay, in which case the benefit of the 2nd WebRTC would be negligible with respect to the effort of making it happen.... maybe we need to measure this (I haven't found any obvious latency numbers on metrics.torproject.org), or we could just decide not to bother and close this.

@arlolra
Copy link
Collaborator

@arlolra arlolra commented Mar 22, 2018

@arlolra arlolra closed this Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment