rtsp-client: Fix iteration

Wouldn't even enter the code block otherwise (i++ was used as the check
and not the postfix).

client: add vmethod to configure media and streams

Implement a vmethod that can be used to configure the media and the
streams based on the current context. Handle the blocksize handling in
the default handler.

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

Make git ignore more unit test binaries

rtsp-server: add padding to many public structures

Not mini objects though, since they are not subclassable
anyway, nor kept on the stack or inlined in a structure.

media: add new create_rtpbin vmethod

* gst/rtsp-server/rtsp-media.[ch]: add new create_rtpbin vmethod.

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

tests: fix memory leak, free test's thread pool

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

stream-transport: free url in finalize

media: also do state change in suspended state

media: also handle prepare and range in suspended state

When we are suspended, we are already prepared.
We can get the range in the suspended state.

check: add test for uri in setup

Added unit tests for the new functionality in GstRTSPStreamTransport.

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

client: store setup uri and use in PLAY response

Store the uri used when doing the setup and use that in the PLAY
response.

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

stream-transport: add method to get/set url

client: suspend after SDP and unsuspend before PLAYING

Based on patches by Ognyan Tonchev <ognyan@axis.com>

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

media: add suspend modes

Add support for different suspend modes. The stream is suspended right after
producing the SDP and after PAUSE. Different suspend modes are available that
affect the state of the pipeline. NONE leaves the pipeline state unchanged and
is the current and old behaviour, PAUSE will set the pipeline to the PAUSED
state and RESET will bring the pipeline to the NULL state.
A stream is also unsuspended when it goes back to PLAYING, for RESET streams,
this means that the pipeline needs to be prerolled again.

Base on patches by Ognyan Tonchev <ognyan@axis.com>

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

media: start live streams in blocked state

Start live streams in the blocked state and make them preroll using the
messages. This ensure that no data is played by the sink until we explicitly
unblock the stream right before going to PLAYING.

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

media: refactor starting and waiting for preroll

Based on patches from Ognyan Tonchev <ognyan@axis.com>

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

stream: add API to block streams

Add an API to block on the streams and make it post a message.

Based on patch by Ognyan Tonchev <ognyan@axis.com>

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

docs: Specify the override file

Even if it's empty (for now) it avoids make distcheck complaining

media: move default implementations to where they are used

media: take the right lock in gst_rtsp_media_set_pipeline_state()

We need to take the state_lock when calling this method.

media: handle add-added on non-bins too

Handle dynamic payloaders that are not bins, as used in the unit-test.

rtsp-media/-factory: Fix request pad name comments

These must be escaped for gtk-doc to parse the comments without warnings.

rtsp-media: remove transports if media is in error status

* gst/rtsp-server/rtsp-media.c (gst_rtsp_media_set_state): if we are
  trying to change to GST_STATE_NULL and media is in error status, we
  remove all transports.

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

rtsp-media: use element metadata to find payloader

Use the element metadata to find the payloader instead of checking
for the base class.

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

rtsp-stream: add getter for payload type

* gst/rtsp-server/rtsp-stream.c: add new method gst_rtsp_stream_get_pt.

* gst/rtsp-server/rtsp-media.c (pad_added_cb): find real payloader
  element and create the stream with this one instead of the dynpay%d
  element.

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

rtsp-*: Refer to NULL as a constant in comments

Plus one typo fix.

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

rtsp-*: Fix type name typos in comments

  * rtsp-auth: Refer to GstRTSPToken, not GstRTSPtoken
  * rtsp-auth: Refer to part of constant name as text
  * rtsp-auth/-permissions/-token: Refer to Permissions not Permission
  * rtsp-session-media: Fix GstRTSPSessionMedia typo
  * rtsp-stream: Fix typo when refering to GstBin

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

docs: Improve documentation

  * Include annotation-glossary to quiet gtk-doc
  * Rename remaining ClientState -> Context
  * Rename object hierarchy file
  * Remove stale chapter references
  * Add missing function and object references
  * Include missing GstRTSPAddressPoolResult

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

rtsp-server: sprinkle some allow-none annotations for g-i

stream: add method to filter transports

Add a method to safely iterate and collect the stream transports

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

rtsp: allow NULL func in filters

Passing a null function make the filters return a list of
refcounted objects.

address-pool: fix address increment

Use a guint instead of guint8 to increment the address. It's still not
completely correct because a guint might not be able to hold the complete
address range, but that's an enhacement for later.
Add unit test to test improved behaviour.

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

client: allow absolute path in requests

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

client: make make_path_from_uri a vmethod

stream: Add functions to get rtp and rtcp sockets

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

context: defing a GType for the context

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

Fixed several GIR warnings

auth: small typos

tests: Add unit tests for token

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

token: Validate args for gst_rtsp_token_is_allowed

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

token: Fix bug when creating empty token

We always want to have a valid GstStructure in the token.

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

thread-pool: avoid race in shutdown

If we call g_main_loop_quit before the thread has entered g_main_loop_run, we
don't actually stop the mainloop ever. Solve this race by adding an idle source
to the mainloop that calls the _quit. This way we immediately exit the mainloop
if quit was called before we started it.

tests: Add unit tests for permissions

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

tests: Test mediafactory permissions

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

permissions: Fix refcounting when adding/removing roles

Previously a role that was removed was unreffed twice, and when
replacing an existing role the replaced role was freed while still being
referenced. Both bugs are now fixed.

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

tests: Check gst_rtsp_url_parse return value

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

Automatic update of common submodule

From 865aa20 to dbedaa0

rtsp-server: Fix socket leak

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

rtsp-session-pool: Make sure session IDs are properly URI-escaped

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

examples: fix compilation when WITH_AUTH is defined

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

gitignore: Add new test binary

thread-pool: Add unit test for the thread pools

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

thread-pool: Fix thread leak when reusing threads

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

tests: fixed racy behavior in rtspserver tests

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

tests: Improve address pool unit tests

Add a range with mixed IPV4 and IPV6 addresses to pool.
Get an IPV4 address from an IPV6-only pool.
Get an IPV6 address from an IPV4-only pool.
Reserve a IPV6 address from an IPV4-only pool.
Check for unicast addresses in multicast-only pool.
Check for unicast addresses in uni-/multicast-mixed pool.

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

client: append query string in PAUSE/PLAY/TEARDOWN as well

client: Add query to control path

If the SETUP url contains a query it must be appended to the control
path so that it matches any already created stream in the media. The
query will also be appended to the session media path.

rtsp-media: remove old line

stream: Correct control comparison

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

media: Check dynamically if the pipeline supports seeking

We should not depend on whether or not the pipeline state change
returned NO_PREROLL or not. A media could dynamically change its
element and switch from seekable to non seekable so it's best to test
the seekable nature of the pipeline dynamically when we try to do a seek.

media: Return FALSE if seeking is not supported

rtsp-media: don't seek accurate by default

Accurate seeking is perhaps a little overkill in the most common situation and
causes some formats (mp3) over slow media to seek extremely slowly.

tests: fix unit test

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

client: Reply 400 if media cannot be constructed

Reply 400 Bad Request instead of 503 Service Unavailable if media
cannot be constructed in SETUP.

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

client: Send setup reply once only

If find_media() failed in handle_setup_request() two replies was sent.

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

Automatic update of common submodule

From 6b03ba7 to 865aa20

server: Emit client-connected signal earlier

Emit client-connected before the client ref is given to a GSource,
otherwise client-connected can be emitted after the client object has
been freed.

addresspool: return reason of failure

Let gst_rtsp_address_pool_reserve_address() return the reason why
the address could not be reserved.

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

autogen.sh: Sync behaviour with other GStreamer modules

Allows building from outside of tree amongst other things

Automatic update of common submodule

From b613661 to 6b03ba7

Automatic update of common submodule

From 74a6857 to b613661

Automatic update of common submodule

From 01a7a46 to 74a6857

client: Do not read beyond end of path string

If the setup was done without a control url, make sure we don't try to read the
non-existing control string and crash.

client: Fix RTPInfo header

Refactor the method to make the content_base.
Use the content-base and the control url to construct the RTPInfo
url.

client: map url to path only in describe

Only map the request url to a path in the DESCRIBE method. The SDP then
contains the base and control urls that should be used to SETUP/PAUSE/
PLAY/TEARDOWN the media.

Revert "client: map URL to path in requests"

This reverts commit e3fded2cec897a2ec003450607b916cc1601fd2d.

This is not correct, we only remap the URL to a path in DESCRIBE, the SDP then
contains the base and control urls which are used in the SETUP, PLAY,
PAUSE and TEARDOWN requests.

client: map URL to path in requests

mount-points: make vmethod to make path from uri

Make a vmethod to transform an url into a path. The path is then used to lookup
the factory. This makes it possible to also use other bits of the url, such as
the query parameters, to locate the factory.

thread-pool: Add cleanup to wait for the threadpool to finish

Also fix race condition if two threads are asking for the first
thread from the thread pool at once. This would case two internal
GThreadPools to be created.

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

client: free threadpool

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

mountpoints tests: unref matched factories

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

media tests: unref thread pool and caps

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

auth, media, media-factory: unref permissions

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

Makefile: add rule for appsrc example

tests: add appsrc example

Add an example on how to use appsrc to feed the server pipeline with data.

rtsp-client: remove query part from content-base string

Make sure that after the control url has been resolved, it's
not a part of the query-string.

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

client: don't check url in response

There is no url or method in the response to check

Add handle-response signal for when we receive a GET_PARAMETER response

Fix gst_rtsp_server_client_filter, using wrong variable type

rtsp-media-factory-uri: check AAC properly for whether it's parsed or not

For AAC we need to check for framed=true instead of parsed=true.

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

stream: optimize pipeline for protocols

When TCP is not an allowed protocol for the stream, avoid creating the
appsrc/appsink/queue and tee elements.

media: set protocols on streams

client: use protocols supported by stream

media-factory: allow all protocols

media: configure protocols in new streams

stream: add protocols property

rtsp-media: send state in "new-state" signal

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

build: add subdir-objects to AM_INIT_AUTOMAKE

Fixes warnings with automake 1.14

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

server: add method to iterate clients of server

Add vmethod for rtsp-media subclass to access rtpbin