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

No format parameters sent for video at DESCRIBE #24

Closed
Fruchtzwerg94 opened this issue Feb 19, 2021 · 2 comments · Fixed by #29
Closed

No format parameters sent for video at DESCRIBE #24

Fruchtzwerg94 opened this issue Feb 19, 2021 · 2 comments · Fixed by #29
Labels
bug Something isn't working enhancement New feature or request question Further information is requested

Comments

@Fruchtzwerg94
Copy link
Contributor

Fruchtzwerg94 commented Feb 19, 2021

Publishing an RTSP stream includes the following DESCRIBE response for my video channel:

m=video 0 RTP/AVP 96
a=rtpmap:96 H264/90000
a=control:track0

The SDP: Session Description Protocol --> Section 6 also describes format parameters like

a=fmtp:<format> <format specific parameters>

This attribute allows parameters that are specific to a particular format to be conveyed in a way that SDP does not have to understand them. The format must be one of the formats specified for the media. Format-specific parameters may be any set of parameters required to be conveyed by SDP and given unchanged to the media tool that will use this format. At most one instance of this attribute is allowed for each format.

which may are required for some players to identify parameters like SPS and PPS to play the stream correctly. See: sprop-parameter-sets

Is it possible to add the format parameters to the DESCRIBE response?

For AAC audio this is already implemented:

sprintf(config, "%02x%02x", (uint8_t)((profile+1) << 3)|(index >> 1), (uint8_t)((index << 7)|(channels_<< 3)));

but its basically missing at H264:

return string("a=rtpmap:96 H264/90000");

@brodi1 brodi1 added bug Something isn't working enhancement New feature or request question Further information is requested labels Feb 20, 2021
@iamscottxu
Copy link
Owner

https://github.com/iamscottxu/obs-rtspserver/releases/tag/v1.4.1-rc1

@Fruchtzwerg94
Copy link
Contributor Author

Wow, that was fast. I've tested it, works for me!
Keep on doing like this, thanks!

@iamscottxu iamscottxu linked a pull request Mar 9, 2021 that will close this issue
Merged
@iamscottxu iamscottxu mentioned this issue Mar 9, 2021
Merged
iamscottxu added a commit that referenced this issue Mar 9, 2021
Bug Fix: Add format parameters for video in SDP. (#24)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants