webrtc-stats icon indicating copy to clipboard operation
webrtc-stats copied to clipboard

RTCIceCandidate: add tcpType

Open fippo opened this issue 4 years ago • 4 comments

as as defined as in webrtc-pc

Not terribly important but i wanted it to show a nice candidate grid from statistics


Preview | Diff


Preview | Diff

fippo avatar Nov 05 '21 18:11 fippo

@dontcallmedom can you generate an IPR link please and take a look at the respec woes? Worked with 27 here, breaks with 28 now. Probably need to blow some dust off ;-)

fippo avatar Nov 08 '21 09:11 fippo

Makes sense

henbos avatar Feb 22 '22 14:02 henbos

actually the inconsistency is a bit bigger: https://w3c.github.io/webrtc-pc/#rtcicecandidate-interface is missing: url, relayProtocol

https://w3c.github.io/webrtc-stats/#icecandidate-dict* is missing: foundation, component, relatedAddress, relatedPort, usernameFragment

I don't have a use-case for the missing candidate stats (apart from making the grid in webrtc-internals nicer) but knowing the relayProtocol and url of a gathered candidate is potentially useful (see e.g. here)

Will update this after testing how hard that is to implement

fippo avatar Aug 15 '22 06:08 fippo

aha... url is apparently surfaced in onicecandidate: https://w3c.github.io/webrtc-pc/#rtcpeerconnectioniceevent I don't think that is implemented by anyone even?

fippo avatar Aug 15 '22 21:08 fippo

bump

fippo avatar Sep 26 '22 17:09 fippo