Skip to content

Commit

Permalink
Merge pull request #2996 from w3c/transport-qs-parsing
Browse files Browse the repository at this point in the history
Complete ice server url validation
  • Loading branch information
henbos authored Oct 24, 2024
2 parents 9cdcb10 + 22f85c4 commit e00c759
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
8 changes: 7 additions & 1 deletion amendments.json
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,14 @@
"validate-ice-server-algo": [
{
"description": "Use the url spec to parse ice server urls",
"pr": 2853,
"pr": [2853, 2996],
"type": "correction",
"tests": [
"webrtc/RTCConfiguration-iceServers.html"
],
"testUpdates": [
"web-platform-tests/wpt#47959"
],
"status": "candidate",
"id": 33
}
Expand Down
2 changes: 1 addition & 1 deletion webrtc.html
Original file line number Diff line number Diff line change
Expand Up @@ -3143,7 +3143,7 @@ <h4>
For "turn" and "turns" schemes, this and the steps below validate
[[!RFC7065]] section 3.1.</p>
</li>
<li><p>TODO: validate ?transport=udp|tcp (see <a href="https://github.com/w3c/webrtc-pc/pull/2996">PR #2996</a>)</p></li>
<li><p>If <var>parsedURL</var>'s [=url/query=] is non-null and if <var>parsedURL</var>'s [=url/query=] is different from either <code>"transport=udp"</code> or <code>"transport=tcp"</code>, [=exception/throw=] a "{{SyntaxError}}" {{DOMException}}.</p></li>
<li>
<p>
If <var>parsedURL</var>'s' [=url/scheme=] is <code>"turn"</code> or
Expand Down

0 comments on commit e00c759

Please sign in to comment.