Wrong RTP header extension parsing when it includes a direction
See original GitHub issueThe following valid RTP header extension attribute:
a=extmap:1/sendonly urn:ietf:params:rtp-hdrext:ssrc-audio-level
produces the following parsed object:
{
uri: "urn:ietf:params:rtp-hdrext:ssrc-audio-level",
value: "1/sendonly"
}
It’s wrong. value
should be 1
(numeric) and a new direction
key should be sendonly
.
Issue Analytics
- State:
- Created 7 years ago
- Comments:6 (3 by maintainers)
Top Results From Across the Web
A General Mechanism for RTP Header Extensions RFC 8285
Standards Track [Page 2] RFC 8285 RTP Header Extensions October 2017 1. ... A direction attribute MAY be included in an "extmap"; without...
Read more >parse/read video orientation from extended headers #278
Is there a reason reading the video orientation from the rtp extended header is not included in mediasoup v2; All the other extended...
Read more >RFC 3550: RTP: A Transport Protocol for Real-Time Applications
To cope with these impairments, the RTP header contains timing information and a sequence number that allow the receivers to reconstruct the timing...
Read more >rtp.h File Reference - Janus WebRTC Server - Meetecho
Helper to parse a dependency descriptor RTP extension (https://aomediacodec.github.io/av1-rtp-spec/#dependency-descriptor-rtp-header-extension) ...
Read more >Question on the structure of RTP Extension headers as ...
This probably dates back to RFC 3550. Specifying the length field explicitly like this allows clients that do not understand extensions to ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Yeah, ended up bumping the major just to be sure. Hate it when patch bumps break my code underneath me 😃
sdp-transform@2.0.0 out now.
Thanks again.
Great, thanks!