Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

implement sink IDRRequest handling #150

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

michaelolbrich
Copy link
Contributor

I've tested this with several devices. Two things should be noted:

  1. some devices really don't like it if they get a IDRRequest before they replied to the last one and stop sending RTSP messages.
    I've added the IDRRequestDone() callback to be able to handle this in my application, but maybe we should prevent the second request inside libwds?
  2. The default timeout is not always sufficient. I have a local hack that reimplements GetResponseTimeout() with a 30 second timeout to work around this issue. I'm not sure how to handle this correctly.

@@ -155,6 +155,8 @@ class SinkMediaManager : public MediaManager {
* @return connector type. @see ConnectorType
*/
virtual ConnectorType GetConnectorType() const = 0;

virtual void IDRRequestDone(bool success) = 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls provide description for the newly added method

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants