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

WebRTC echo server #36487

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

WebRTC echo server #36487

wants to merge 7 commits into from

Conversation

fippo
Copy link
Contributor

@fippo fippo commented Oct 17, 2022

No description provided.

@fippo fippo changed the title Webrtc echo WebRTC echo server Jan 26, 2023
@fippo
Copy link
Contributor Author

fippo commented Jan 26, 2023

@foolip: @alvestrand says you might be able to help.
We discussed the topic of wire-level WebRTC testing in the WG back in February 2021, see minutes or recording for the motivation.
The energy for this has been pretty low sadly (hence the almost two years delay) but if we can get the basic infrastructure in place it might be enough to let Harald ask for tests which usually motivates folks.

This is modelled after the aioquic-based server which was super helpful to follow (aiortc and aioquic being written by the same person helped too).

What would it take to land this?

Possibly without any active tests first and then we iterate in the chromium infrastructure because how stable such a test is... remains to be seen 🤞 (I still remember how much effort went into this back in the day)

@henbos
Copy link
Contributor

henbos commented May 30, 2023

While not up to speed on the capabilities of this server, having an echo server sounds like something that could help with improving test coverage of webrtc stats (w3c/webrtc-stats#760).

Today we mostly just assert the presence of metrics, not what any of the counters should be...

@foolip is adding a pyhton server to WPTs something that can/should be done?

@foolip
Copy link
Member

foolip commented Sep 15, 2023

@henbos yes, I think such a server would be accepted in this project. It's a bit of work and requires going through the process at https://github.com/web-platform-tests/rfcs, but the concept should be uncontroversial as it's similar to how we test HTTP, WebTransport, etc.

@fippo
Copy link
Contributor Author

fippo commented Sep 27, 2023

since I just struggled remembering here is the tldr for getting this to run:

python -m venv env
env\scripts\activate
pip install -r tools\webrtc\requirements.txt
python wpt serve --webrtc

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

Successfully merging this pull request may close these issues.

3 participants