adaptivedemux2: Don't blindly set the main manifest URI as referer
authorEdward Hervey <edward@centricular.com>
Wed, 15 Feb 2023 17:06:36 +0000 (18:06 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Tue, 18 Jul 2023 14:00:49 +0000 (16:00 +0200)
There's no guarantee it will *actually* be the URI which refered to what we are
downloading. It could be a stream URI or anything else.

Instead of putting something wrong, put no (specific) referer as a better choice

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/5057>

subprojects/gst-plugins-good/ext/adaptivedemux2/gstadaptivedemux-stream.c

index 217d78e..fefa656 100644 (file)
@@ -1427,7 +1427,7 @@ gst_adaptive_demux2_stream_begin_download_uri (GstAdaptiveDemux * demux,
   }
 
   if (!downloadhelper_submit_request (demux->download_helper,
-          demux->manifest_uri, DOWNLOAD_FLAG_NONE, request, NULL))
+          NULL, DOWNLOAD_FLAG_NONE, request, NULL))
     return GST_FLOW_ERROR;
 
   stream->download_active = TRUE;