mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-11-27 12:11:13 +00:00
rtspsrc: Retry connection if tunneling needs authentication
Leverage response from gst_rtsp_connection_connect_with_response to determine if the connection should be retried using authentication. If so, add the appropriate authentication headers based upon the response and retry the connection. https://bugzilla.gnome.org/show_bug.cgi?id=749596
This commit is contained in:
parent
4735d2a9a5
commit
4718870959
1 changed files with 76 additions and 50 deletions
|
@ -342,6 +342,8 @@ static gboolean gst_rtspsrc_stream_push_event (GstRTSPSrc * src,
|
|||
GstRTSPStream * stream, GstEvent * event);
|
||||
static gboolean gst_rtspsrc_push_event (GstRTSPSrc * src, GstEvent * event);
|
||||
static void gst_rtspsrc_connection_flush (GstRTSPSrc * src, gboolean flush);
|
||||
static GstRTSPResult gst_rtsp_conninfo_close (GstRTSPSrc * src,
|
||||
GstRTSPConnInfo * info, gboolean free);
|
||||
|
||||
typedef struct
|
||||
{
|
||||
|
@ -4491,19 +4493,26 @@ gst_rtsp_conninfo_connect (GstRTSPSrc * src, GstRTSPConnInfo * info,
|
|||
gboolean async)
|
||||
{
|
||||
GstRTSPResult res;
|
||||
|
||||
GstRTSPMessage response;
|
||||
gboolean retry = FALSE;
|
||||
memset (&response, 0, sizeof (response));
|
||||
gst_rtsp_message_init (&response);
|
||||
do {
|
||||
if (info->connection == NULL) {
|
||||
if (info->url == NULL) {
|
||||
GST_DEBUG_OBJECT (src, "parsing uri (%s)...", info->location);
|
||||
if ((res = gst_rtsp_url_parse (info->location, &info->url)) < 0)
|
||||
goto parse_error;
|
||||
}
|
||||
|
||||
/* create connection */
|
||||
GST_DEBUG_OBJECT (src, "creating connection (%s)...", info->location);
|
||||
if ((res = gst_rtsp_connection_create (info->url, &info->connection)) < 0)
|
||||
goto could_not_create;
|
||||
|
||||
if (retry) {
|
||||
gst_rtspsrc_setup_auth (src, &response);
|
||||
}
|
||||
|
||||
g_free (info->url_str);
|
||||
info->url_str = gst_rtsp_url_get_request_uri (info->url);
|
||||
|
||||
|
@ -4540,19 +4549,34 @@ gst_rtsp_conninfo_connect (GstRTSPSrc * src, GstRTSPConnInfo * info,
|
|||
GST_ELEMENT_PROGRESS (src, CONTINUE, "connect",
|
||||
("Connecting to %s", info->location));
|
||||
GST_DEBUG_OBJECT (src, "connecting (%s)...", info->location);
|
||||
if ((res =
|
||||
gst_rtsp_connection_connect (info->connection,
|
||||
src->ptcp_timeout)) < 0)
|
||||
goto could_not_connect;
|
||||
res = gst_rtsp_connection_connect_with_response (info->connection,
|
||||
src->ptcp_timeout, &response);
|
||||
|
||||
info->connected = TRUE;
|
||||
if (response.type == GST_RTSP_MESSAGE_HTTP_RESPONSE &&
|
||||
response.type_data.response.code == GST_RTSP_STS_UNAUTHORIZED) {
|
||||
gst_rtsp_conninfo_close (src, info, TRUE);
|
||||
if (!retry)
|
||||
retry = TRUE;
|
||||
else
|
||||
retry = FALSE; // we should not retry more than once
|
||||
} else {
|
||||
retry = FALSE;
|
||||
}
|
||||
|
||||
if (res == GST_RTSP_OK)
|
||||
info->connected = TRUE;
|
||||
else if (!retry)
|
||||
goto could_not_connect;
|
||||
}
|
||||
} while (!info->connected && retry);
|
||||
gst_rtsp_message_unset (&response);
|
||||
return GST_RTSP_OK;
|
||||
|
||||
/* ERRORS */
|
||||
parse_error:
|
||||
{
|
||||
GST_ERROR_OBJECT (src, "No valid RTSP URL was provided");
|
||||
gst_rtsp_message_unset (&response);
|
||||
return res;
|
||||
}
|
||||
could_not_create:
|
||||
|
@ -4560,6 +4584,7 @@ could_not_create:
|
|||
gchar *str = gst_rtsp_strresult (res);
|
||||
GST_ERROR_OBJECT (src, "Could not create connection. (%s)", str);
|
||||
g_free (str);
|
||||
gst_rtsp_message_unset (&response);
|
||||
return res;
|
||||
}
|
||||
could_not_connect:
|
||||
|
@ -4567,6 +4592,7 @@ could_not_connect:
|
|||
gchar *str = gst_rtsp_strresult (res);
|
||||
GST_ERROR_OBJECT (src, "Could not connect to server. (%s)", str);
|
||||
g_free (str);
|
||||
gst_rtsp_message_unset (&response);
|
||||
return res;
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue