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/6926>
This commit is contained in:
Sebastian Dröge 2024-05-26 13:00:02 +03:00 committed by GStreamer Marge Bot
parent 2b76f36843
commit 966c39b92e

View file

@ -7866,7 +7866,7 @@ gst_rtspsrc_setup_streams_start (GstRTSPSrc * src, gboolean async)
* https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1447
*/
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;
gst_rtsp_message_unset (&request);