[Bug 777222] New: souphttpsrc: redirect uri is never set

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=777222

            Bug ID: 777222
           Summary: souphttpsrc: redirect uri is never set
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: rawoul at gmail.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Created attachment 343438
  --> https://bugzilla.gnome.org/attachment.cgi?id=343438&action=edit
souphttpsrc: properly track redirections

Hi,

In 1.10 and git master I can see that some HLS streams do not play. In this
case this is due to a redirect in the GET request for the master playlist. The
variant playlist URIs are relative, and the base URI is set incorrectly to the
master playlist URI, instead of the URI set in the Location header from the
redirect reply.

The code to handle the base URI is correct in adaptivedemux/hlsdemux, using an
URI query to the source element to retrieve the download URI, possible redirect
URI and the permanent flag (301 or 302 code).

However souphttpsrc never sets the redirect URI. In the current code libsoup is
configured to transparently handle redirects, instead of letting the caller
handle the 30x code.

I'm attaching a tentative patch that works for me, but I'm not confident that
setting the src->ret code like I did here is correct.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.