rtspsrc: Don't try the SETUP workaround for broken servers with absolute control URIs

Previously only control URIs that started with "rtsp://" were ignored
but it makes more sense to ignore all absolute URIs.

gst_uri_is_valid() conveniently checks for exactly that.

See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/3563

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6982>
This commit is contained in:
Sebastian Dröge 2024-05-26 13:00:02 +03:00 committed by Backport Bot
parent 03b6efcaf5
commit dfc03b9a2e

View file

@ -7838,7 +7838,7 @@ gst_rtspsrc_setup_streams_start (GstRTSPSrc * src, gboolean async)
* https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1447 * https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1447
*/ */
if (!tried_non_compliant_url && stream->control_url if (!tried_non_compliant_url && stream->control_url
&& !g_str_has_prefix (stream->control_url, "rtsp://")) { && !gst_uri_is_valid (stream->control_url)) {
const gchar *base; const gchar *base;
gst_rtsp_message_unset (&request); gst_rtsp_message_unset (&request);