- Notifications
You must be signed in to change notification settings - Fork115
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Make RTCRtpHeaderExtensionCapability.uri required#2841
Conversation
@dontcallmedom is this an amendment or an editorial? Inside webrtc-pc, this dictionary is only used as output meaning it doesn't change any behavior. But in webrtc-extension we start taking this dictionary as input, and do care about "required" |
it's very much in the gray zone; I would err on the side of being conservative and document it as an amendment. |
OK. I'm unfamiliar with the amendment process - should I do something to unblock this PR? |
I'll take care of it |
ec3fe8c
tod95d66c
CompareThanks! Can I merge now? |
Good to go from my perspective |
Fixes#2840
Preview |Diff