[Bug 777222] souphttpsrc: redirect uri is never set

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #343438|none                        |reviewed
             status|                            |

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 343438:
 --> (https://bugzilla.gnome.org/review?bug=777222&attachment=343438)

::: ext/soup/gstsouphttpsrc.c
@@ +1064,3 @@
           msg->status_code, src->redirection_uri, src->redirection_permanent);
+
+      soup_session_redirect_message (src->session, msg);

We should probably first check with soup_session_would_redirect() if there's an
actual redirect... and then only call soup_session_redirect_message() if
src->automatic_redirect==TRUE

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