Problem with multipart boundaries in gstreamer applications.

[rvbglas]

0. Reviewed guide and contributing documents? (Yes/No): Yes
1. version [x.y.z, hash, other]: 4.1.1
2. installed as a package or compiled from sources [deb, rpm, git, other]: compiled from git sources
3. standalone or part of third party [motion, MotionEyeOS, other]: standalone
4. video stream source [V4L (card or USB), net cam (mjpeg, rtsp, other), mmal]: mjpeg + rtsp
5. hardware [x86, ARM, other]: amd64
6. operating system [16.04, Stretch, etc, FreeBSD, other]: Ubuntu 16.04

After migration to libmicrohttpd motion webstreams cannot be used in gstreamer apps, but still viewable in browser.

>gst-launch-1.0 -v souphttpsrc location=http://127.0.0.1:8082 ! multipartdemux ! jpegdec ! fakesinkSetting pipeline to PAUSED ...
Pipeline is PREROLLING ...
ERROR: from element /GstPipeline:pipeline0/GstMultipartDemux:multipartdemux0: Could not demultiplex stream.
Additional debug info:
multipartdemux.c(503): multipart_parse_header (): /GstPipeline:pipeline0/GstMultipartDemux:multipartdemux0:
Boundary not found in the multipart header
ERROR: pipeline doesn't want to preroll.
Setting pipeline to NULL ...
Freeing pipeline ...

4.1 works normally.

[Mr-Dave]

PR created. My testing indicates it resolves issue but I would appreciate it if you could also validate

[tosiara]

I can't verify the OP's command, it fails for me with WARNING: erroneous pipeline: no element "fakesinkSetting"
But I have checked the PR and the --BoundaryString now appears in the headers

[Mr-Dave]

You need to revise it to be just fakesink

I'd note that the--BoundaryString was there in the previous code. It was just the terminator for the data versus now there needs to be another message sent to terminate the message. This was how 4.1 did the stream processing, but it feels wrong.

If anyone has a RFC citation on where to place the boundary string (end of current message or start of next message), please comment.