rtsp-client: Fix handling of keep-alive GET_PARAMETER/SET_PARAMETER

While they logically have 0 bytes length, GstRTSPConnection is appending
a '\0' to everything making the size be 1 instead.

meson: bump version

rtsp-session: Only remove deprecated API if requested to do so, not just when disabling

gst_rtsp_session_is_expired() and gst_rtsp_session_next_timeout() were
affected.

Back to development

Release 1.11.1

rtsp-stream: corrected if-statement in _get_server_port()

This bug was accidentally introduced while fixing a segfault
in _get_server_port() function.

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

rtsp-stream: fixed segmenation fault in _get_server_port()

Calling function gst_rtsp_stream_get_server_port() results in
segmenation fault in the RTP/RTSP/TCP case.
Port that the server will use to receive RTCP makes only
sense in the UDP case, however the function should handle
the TCP case in a nicer way.

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

dosc: Fix a little typo

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

meson: generate pkg-config -uninstalled pc files

Generating those files is useful for users building the GStreamer stack
using meson and having to link it to another project which is still
using the autotools.

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

pkgconfig: fix -uninstalled pc file

pcfiledir was never defined so the paths were wrong.

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

rtsp-stream: Fixed TCP transport case

Make sure that the appsink element is actually added to
the bin before trying to link it with the elements in it.

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

Remove generated .spec file

Likely extremely bitrotten, and we should not ship this anyway.

Automatic update of common submodule

From f980fd9 to 39ac2f5

media: Fix pt map caps

Since decryption is handled within rtpbin, all outcoming stream
caps will be application/x-rtp (i.e. regular rtp)

Fixes RECORD with SRTP streams

media-factory: Create media objects with the proper transport mode

The function called immediately afterwards (collect_streams()) will
need it to work properly

rtsp-auth: Don't remove digest-auth nonces that already/still have a client connected

rtsp-media-factory: Don't create a pipeline for the media pipeline string

We're going to put a pipeline into a pipeline otherwise, which is not
exactly ideal.

media: Fix race condition around finish_unprepare() if called multiple time

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

rtspclientsink: Don't leave stale pointer after unref

Fix a warning on shutdown - don't keep a pointer to an
alread-unreffed object.

common: use https protocol for common submodule

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

stream: block the output of rtpbin instead of the source pipeline

85c52e194bcb81928b96614be0ae47d59eccb1ce introduced a more correct
detection of the srtp rollover counter to add to the SDP.

Unfortunately, it was incomplete for live pipelines where the logic
blocks the source bin before creating the SDP and thus would never have
the necessary informaiton to create a correct SDP with srtp encryption.

Move the pad blocks to rtpbin's output pads instead so that the
necessary information can be created before we need the information for
the SDP.

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

rtsp-client: add IDLE timeout, before session exists

The RTSP server will not timeout an idle RTSP connection
(note this is different from doing timeout on a RTSP
session).

At least for Apache this is a problem when running RTSP over
HTTPS since it uses one of the threads (there is a rather
limited number) that are available for handling requests.

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

.gitignore more

rtsp-stream: Set close-socket FALSE on UDP src:es

With this RTSP server can use the sockets independent on the udpsrc
state.
When the udp src is finalized it will unref socket and when g_socket
is finalized the socket will be closed.

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

rtspclientsink: Move to new helper function to parse authentication responses

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

rtsp-auth: Add support for Digest authentication

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

Enable building with MSVC

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

meson: gstreamer gst_check_dep does not exist on windows

client: update do_send_message to match type GstRTSPClientSendFunc

This type mismatch fails building with MSVC

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

rtsp-sdp: Fix indentation

rtsp-media: Only signal "new-state" if the state has actually changed

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

client: emit signal in the beginning of each rtsp request

These signals let the application validate the requests, configure the
media/stream in a certain way and also generate error status code in
case of error or bad request.

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

meson: update version

Back to development

Release 1.10.0

tests: try to avoid using the same ports in different tests

Causes problems with client multicast tests otherwise if
tests are run in parallel.

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

tests: client: use fail_unless_equals_foo() for better failure reporting

rtsp-client: Session filter in unwatch session

Call session filter with filter_session_media as paramer in
client_unwatch_session if using drop_backlog = FALSE.

In client_unwatch_session its allowed to grow the watchs backlog.
If using drop_backlog = FALSE and the backlog is full it will cause
a deadlock when setting session media state to NULL
if the backlog is not allowed to grow.

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

meson: add fallbacks for gst modules

For gst-all.

rtsp-client: Fix factory leaking in find_media() in error cases

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

stream: Fix randomly missing streams from SDP with dynamic elements

When using dynamic elements, gst_rtsp_stream_join_bin() is called from
"pad-added" signal. In that case priv->srcpad could already have its caps,
and they'll be sent to priv->send_src[0] pad. That means that when it
connects "notify::caps" signal, that pad could already have received its
caps and the signal won't be emitted anymore.

In that case priv->caps stay to NULL and when building the SDP that stream
gets ignored. Leading to missing video or audio when playing in client side.

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

meson: update version

Release 1.9.90

rtsp-server: Hint that set_multicast_iface expects the name of the interface

To prevent any possibly confusion with IPs or anything else.

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

rtsp-media: Call g_free() instead of g_object_unref() on multicast-iface strings

https://bugzilla.gnome.org/show_bug.cgi?id=763000#c5

configure: Depend on gstreamer 1.9.2.1

Automatic update of common submodule

From b18d820 to f980fd9

Automatic update of common submodule

From 6f2d209 to b18d820

rtsp-stream: Remove unused _locked() variant of a function

It was added during refactoring.

stream: cosmetic cleanup

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

stream: Compare IP addresses case insensitive in more places

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

stream: Fix leaked joined_bin

There is no need to keep a strong ref on it, and _leave_bin() was
setting it to NULL before calling g_clear_object() so it was leaked.

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

rtsp-stream: Compare IP address strings case insensitive

Otherwise IPv6 addresses might fail this comparision.

rtsp-stream: Bind multicast sockets to ANY as before

https://bugzilla.gnome.org/show_bug.cgi?id=766612#c48

rtsp-session: Fix segfault when doing keep-alive after removing the session

If keep-alive happens after removing the session but before finalizing the
stream transport, we would segfault.

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

rtsp-stream: Always create multicast UDP elements if the protocol flag is set

Adding them later will cause deadlocks due to
1) pre-rolling and staying in PAUSED with the unicast/TCP sinks
2) adding the multicast sink
3) waiting for it to get data to preroll again

3) never happens because the queues after the tee are full.

rtsp-stream: Fix up various multicast related issues

tests: Fix compilation

stream: revert back to create udpsrc/udpsink on DESCRIBE for unicast

This is basically reverting changes introduced in commit f62a9a7,
because it was introducing various regressions:

- It introduces a leak of udpsrc elements that got wrongly fixed by adding
  an hash table in commit cba045e. We should have at most 4 udpsrc for unicast:
  ipv4/ipv6, rtp/rtcp. They can be reused for all unicast clients.
- If a mcast client connects, it creates a new socket in SETUP to try to respect
  the destination/port given by the client in the transport, and overrides the
  socket already set on the udpsink element. That means that if we already had a
  client connected, the source address on the udp packets it receives suddenly
  changes.
- If a 2nd mcast client connects, the destination/port in its transport is
  ignored but its transport wasn't updated.

What this patch does:

- Revert back to create udpsrc/udpsink for unicast clients on DESCRIBE.
- Always have a tee+queue when udp is enabled. This could be optimized
  again in a later patch, but is more complicated. If no unicast clients
  connects then those elements are useless, this could be also optimized
  in a later patch.
- When mcast transport is added, it creates a new set of udpsrc/udpsink,
  seperated from those for unicast clients. Since we already support only
  one mcast address, we also create only one set of elements.

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

stream: factor our plug_src function

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

stream: factor out plug_sink function

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

stream: small documentation clarification

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

stream: rename addr_v4/6 to mcast_addr_v4/6 for clarity

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

stream: Keep a ref on joined bin

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

stream: code cleanup

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

stream: small fix in error code path

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

Revert "rtsp-stream: Fix crash on cleanup with shared media and multiple udpsrc"

This partly reverts commit cba045e1b19fad6e689e10206f57903e15f1229a,
but keeps unit tests.

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

Back to development

Release 1.9.2

Add support for Meson as alternative/parallel build system

https://github.com/mesonbuild/meson

build: silence error about pthread for 'make check' in osx

Fixes "clang: error: argument unused during compilation: '-pthread'"

rtsp-client: Fix leaking of media in error cases

With additional fixes by Kseniya Vasilchuk <vasilchukkseniia@gmail.com>
and myself to make the media refcounting a bit easier to follow.

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

rtsp-client: Fix leaking of session in error cases

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

Automatic update of common submodule

From f363b32 to f49c55e

Back to development

Release 1.9.1

configure: Need to add -DGST_STATIC_COMPILATION when building only statically

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

Automatic update of common submodule

From ac2f647 to f363b32

sdp: add rollover counters for all sender SSRC

We add different crypto sessions in MIKEY, one for each sender
SSRC. Currently, all of them will have the same security policy, 0.

The rollover counters are obtained from the srtpenc element using the
"stats" property.

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

docs: fix some typos

g-i: pass compiler env to g-ir-scanner

It's what introspection.mak does as well. Should
fix spurious build failures on gnome-continuous
(caused by g-ir-scanner getting compiler details
via python which is broken in some environments
so passing the compiler details bypasses that).

rtsp-session: RFC2326 does not allow a space between ; and timeout in the Session header

This works with rtspsrc and live555, but fails with e.g. ffmpeg.

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

rtspclientsink: Check return value of sscanf

And just make sure we always have 0/0 if we have an error

CID #1352031

rtsp-stream: Fix crash on cleanup with shared media and multiple udpsrc

 - Unicast udpsrcs are now managed in a hash table. This allows for proper cleanup in with shared streams and fixes a memory leak.
 - Unicast udpsrcs are now properly cleaned up when shared connections exit. See the update_transport() function.
 - Create unit test for shared media.

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

rtsp-stream: Always bind to ANY when address is a multicast address and not only on Windows

For IPv6 addresses, binding to a multicast group does not work on Linux
either. Always bind to ANY and then later join the multicast group.

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

Automatic update of common submodule

From 6f2d209 to ac2f647

rtsp-thread-pool: explained why GSource is a part of ThreadImpl

Clarified why it is necessary to add source information to
GstRTSPThreadImpl. See the reported bug in GLib:
https://bugzilla.gnome.org/show_bug.cgi?id=720186
for more information.

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

examples: Clean up CFLAGS/LDADD even more

The internal .la should come first and is part of LDADD, as is
GST_CFLAGS/LIBS.

examples: Clean up CFLAGS/LDADD to link with the correct versions of all libraries

rtsp-server: Use $(GST_NET_LIBS) / $(GST_NET_CFLAGS)

rtsp-server: Implement clock signalling according to RFC7273

For NTP and PTP clocks we signal the actual clock that is used and signal
the direct media clock offset.

For all other clocks we at least signal that it's the local sender clock.

This allows receivers to know which clock was used to generate the media and
its RTP timestamps. Receivers can then implement network synchronization,
either absolute or at least relative by getting the sender clock rate directly
via NTP/PTP instead of estimating it from RTP timestamps and packet receive
times.

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

rtspclientsink: Add support for setting the multicast interface

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

rtsp-media: Add support for setting the multicast interface

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

rtspclientsink: use new gst_element_class_add_static_pad_template()

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

Back to development

Release 1.8.0

rtsp-stream: Don't set the state of the appsrc from PLAYING to PAUSED again during setup

This would get us NO_PREROLL in the bin again and break seeking.
Thanks to Carlos Rafael Giani for helping to debug this!

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

Release 1.7.91

rtsp-stream: Ensure that the pipeline is live and later-added udpsrcs are syncing the state with the parent bin

Without this, RECORD pipelines are broken because
a) we wait for ASYNC_DONE which never happens anymore because udpsrc would be
added later. Previously it was there earlier and due to NO_PREROLL caused the
pipeline to preroll immediately
b) the udpsrc for the pipeline is added later and never set to PLAYING state,
as the corresponding code previously was only for PLAY pipelines.

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

rtsp-stream: Fix typo in the docstring

gst_rtsp_stream_set_client_side -> gst_rtsp_stream_is_client_side