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

Remove sctp-transport #627

Merged
merged 4 commits into from
Jun 9, 2022
Merged

Remove sctp-transport #627

merged 4 commits into from
Jun 9, 2022

Conversation

henbos
Copy link
Collaborator

@henbos henbos commented Apr 7, 2022

One of the steps needed for #621.

AFTER THIS PR HAS BEEN APPROVED BUT BEFORE LANDING IT:
Create a corresponding webrtc-provisional-stats PR adding these metrics to that document instead.


Preview | Diff

@Orphis
Copy link

Orphis commented Apr 11, 2022

I believe those would still be useful and could be done in Chrome with the new SCTP implementation.

@henbos
Copy link
Collaborator Author

henbos commented Apr 11, 2022

Are you interested in implementing it any time soon? (If not, we can always add them back to the spec at a later date if removed here)

@henbos
Copy link
Collaborator Author

henbos commented Jun 9, 2022

This is now unblocked because we added it to provisional spec: w3c/webrtc-provisional-stats#30

"client",
"server",
"unknown",
<section>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Drive-by indentation fixes)

@henbos henbos merged commit 3809583 into w3c:main Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants