[Bug 653741] [curl] New protocols in curlsink

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=653741
  GStreamer | gst-plugins-bad | unspecified

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.11.x

--- Comment #34 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2012-05-12 11:05:52 UTC ---
Thanks, I've pushed these to master now.

Haven't reviewed in depth, but looks fine at first glance, don't see any reason
not to just push these.

I would like to see the FIXMEs around the properties fixed (sync property names
with souphttpsrc), if still relevant.

What is the purpose of curlfilesink? What can it do that filesink can't do? Why
have it? I'd rather see options like the create-directories added to filesink.

Why are there separate properties for the file-name and the directory
(location)? I noticed the property description refers to 'for the uploaded
images', but not quite sure what it means / how it works.

Can't we stuff that all into location as it's done for other things (even e.g.
splitfilesrc)?


commit 3f20bbdcf4b437bf63a7426a5deac45a3c8267d7
Author: Sebastian Rasmussen <sebrn at axis.com>
Date:   Wed Apr 25 19:46:53 2012 +0200

    curl: new curlsmtpsink element

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

commit 766dd8bb717b1ba5317fbd2c5d7860d6beb34834
Author: Patricia Muscalu <patricia at axis.com>
Date:   Mon Jan 23 09:45:24 2012 +0100

    curl: new curlftpsink element

    http://bugzilla.gnome.org/show_bug.cgi?id=653741

commit f139656becf17dce39be0fe180796794aa3b8875
Author: Patricia Muscalu <patricia at axis.com>
Date:   Mon Jan 23 09:28:10 2012 +0100

    curl: new curlfilesink element

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

commit ae39cbfa577adf48f50f11b67121670275480d92
Author: Patricia Muscalu <patricia at axis.com>
Date:   Mon Jan 23 09:00:47 2012 +0100

    curl: refactor curlsink, rename to curlhttpsink

    Split into base, tls and http classes.

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

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.