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

small documentation fix

Do not take range header if range is invalid

media: add docs for new method

Add API to rtsp-media set the pipeline's state

Update current position/duration when gst_rtsp_media_get_range_string is called

tests: add some more docs

ClientState -> Context

Rename the clientstate to context and put the code in a separate file.

auth: add support for default token

The default token is used when the user is not authenticated and can be used to
give minimal permissions.

auth: use defines when possible

address-pool: improve docs

permissions: add the role to the copy

permissions: Also copy the roles

permissions: Make it build

docs: small fixes

docs: improve docs

address-pool: cleanups

Remove redundant method, improve docs.

docs: improve docs

permissions: implement _remove_role

permissions: update docs

tests: simplify tests

Client settings are now disabled by default so we don't need an auth
module to disable them.

auth: add default authorizations

When no auth module is specified, use our table of defaults to look up the
default value of the check instead of always allowing everything. This was
we can disallow client settings by default.

README: update readme

thread-pool: add more docs

thread-pool: fix race in thread reuse

If we try to reuse a thread right after we made it stop, we end up using a
stopped thread. Catch this case and only reuse threads that are not stopping.

server: add small debug

client: fix test

Add some permissions to media so we can use the auth and enable
client settings.

client: support pushed context in handle_request

If we already have a pushed state, reuse it and add our own things. This makes
it easier to write tests.

auth: don't auth on methods

Don't authorize on methods anymore but on the resources that we
try to access, this is more flexible.
Move the authorization checks to where they are needed and let the
check return the response on error.

mount-points: add some debug

tests: almost fix test

auth: let the auth module check client_settings

Let the auth module decide if client settings are allowed for the
current client.

token: add method to check boolean permission

token: simplify token constructor

Use variable arguments to make easier API.

media-factory: add convenience API for factory

permissions: simplify API a little

Avoid passing GstStructure in the add_role method, use varargs instead
to construct the structure behind the scenes. We can then also use the
structure name as the role and simplify some more logic.

auth: fix typo

auth: handle unauthorized response

Move handling of the unauthorized response to the auth module, it can add
the appropriate headers to request authorization for the required method
much better than the client.

client: allow for sending any message, not only requests

Change the _send_request() method to _send_message() so that we
can both send requests and replies.

docs: fix docs

auth: move TLS handling to auth module

Remove the TLS settings on the server and move it to the auth module because
that is where security related bits go.

client: add state push/pop

client: add connection to state

mount-points: fix debug

tests: fix media test

thread-pool: we don't require a state

server: let context ref the server

So that we don't risk losing the server object early anc crash.