[Bug 752866] souphttpsrc: check null string before setting proxy

[GStreamer (GNOME Bugzilla)]

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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #308511|none                        |committed
             status|                            |

--- Comment #10 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 308511
  --> https://bugzilla.gnome.org/attachment.cgi?id=308511
souphttpsrc: check null string before setting proxy

Thanks for updating the patch:

commit 8b6a26170346ccf5d01052229a0618d593b910c5
Author: Eunhae Choi <eunhae1.choi at samsung.com>
Date:   Thu Jul 30 11:29:27 2015 +0900

    souphttpsrc: handle empty http proxy string

    1) If the system http_proxy environment variable is not set
    or set to an empty string, we must not set proxy to avoid
    http connection error.

    2) In case of proxy property setting, if user want to clear
    the proxy setting, they should be able to set it to NULL or
    an empty string again, so this is fixed too.

    3) Check if the proxy string was parsed correctly.

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

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