client: cleanups

Rename variables for clarity
Keep media in state when we can

stream: add more support for IPv6

Rename _get_address to _get_multicast_address in GstRTSPStream to
make it clear that this function only deals with multicast.
Make it possible to have both an IPv4 and IPv6 multicast address on
a stream. Give the client an IPv4 or IPv6 address depending on the
address it used to connect to the server.

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

client: fix comment

stream: handle failed port allocation

Allow for ipv4 or ipv6 socket allocations to fail. Only report failure if we
can't allocate any family at all. Also keep track of what port families we
allocated.

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

stream: improve docs

stream-transport: remove old if 0 block

tests: fix tests

gst_rtsp_client_get_uri() has been removed

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

client: add method to filter managed sessions

Add a method to filter the sessions managed by this client connection.

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

client: remove _get_uri() method

Remove the get_uri() method on the client. A client has no uri, the uri
property is an internal property to manage the last cached media for
the client.

media-factory: fix typo

rtsp-media: Do not leak the query in default_query_stop

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

media: don't unlock when conversion fails

Don't unlock the state lock when conversion fails because it was not locked.

Add query_position and query_stop vmethods to rtsp-media

Fix typo in property install for rtsp-media's time-provider

client: clean some variables

Clean some variables and add some guards to _send_request()

Add gst_rtsp_client_send_request API

This makes it possible to send arbitrary messages to a client, such as
SET_PARAMETER or GET_PARAMETER

media: add _get_element() method

Add method to get the element used when creating the media.

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

media: fix docs

stream: allow access to the rtp session

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

dscp qos support in gst-rtsp-stream

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

tests: fix test

Actually do what the comment says. Also keep the old code around, not sure what
should happen when you get a 454 from a TEARDOWN, does it close the connection?
it currently doesn't.

client: also watch newly created session

When we newly created a session, start watching it immediately instead of
on the next request.

tests: add unit test for new-session

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

client: emit new-session when new session is created

Only emit new-session when we created a new session for a client, not when a
client picked up a previous session.

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

client: handle asterisk as path in requests

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

media: handle segment query format mismatch

It's possible that the segment query returns with a different format than what
we asked for, handle this case also.

media: use segment stop in collect_media_stats

Use segment stop instead of duration as range end point.

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

rtsp-media: Do not leak the element in take_pipeline

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

rtsp-client: Make configure_client_transport virtual

This patch makes configure_client_transport virtual. The functionality is
needed to handle some weird clients sending multicast transport settings as url
options.

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

rtsp-client: Make param_set and param_get virtual

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

media: convert_range replaces get_range_times

get_range_times worked for handling UTC ranges for seeks, but we also
need to convert back from NPT to the requested unit in
get_range_string. convert_range is now used for both.

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

sdp: cleanup sdp info

We don't need to pass the proto, we can more easily check a boolean.

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

use 0.0.0.0 or :: for c= line instead of server address

use local address, not remote, in SDP

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

Automatic update of common submodule

From 098c0d7 to 01a7a46

media: possibility to override range time conversion

Make it possible to override the conversion from GstRTSPTimeRange to
GstClockTimes, that is done before seeking on the media
pipeline. Overriding can be useful for UTC ranges, where the default
conversion gives nanoseconds since 1900.

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

rtsp-server: Expose the use_client_settings API

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

rtspstream: handle both ipv4 and ipv6 clients

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

Revert "rtsp-sdp: Parse width/height from caps and set SDP attribute"

This reverts commit 5fd034ff1a517db7f629ffcc3ed16839c61f5c97.

We already have a way to place extra attributes in the SDP by using a string
property with prefix x- or a- in the caps.

Revert "rtsp-sdp: Parse framerate caps field and set SDP attribute"

This reverts commit d6a4dee03642a2d2c05fec4752dc3ccb60b19494.

We already have a way to place extra attributes in the SDP, just make a string
property in the payloader with a- or x- prefix.

rtsp: place a- and x- properties as attributes

application/x-rtp has properties with a- and x- prefixes that should be
placed as attributes in the SDP for the media instead of being added to the
fmtp.

example: add TLS example

server: add support for TLS

Add methods to set and get a TLS certificate.
Add vmethod to configure a new connection. By default, configure the TLS
certificate in a new connection if needed.

server: remove accept_client vmethod

This vmethod is not very useful so remove it.

server: don't crash on NULL GError

rtsp-session-pool: corrected session timeout detection

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

client: improve debug

server: refactor connection setup

Let the server accept the socket connection and construct a GstRTSPConnection
from it. Remove the code from the client and let the client only deal with
a fully configure GstRTSPConnection object.

We will need this later when the server will configure the connection for
TLS.

stream: keep the transport object alive

Keep the transport object alive while we have it as qdata on the
source.

rtsp-server: Do not crash on nmapping of server

* generate error when gst_rtsp_connection_accept fails
* do not stop accepting incoming connections because
  accepting a client fails

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

rtsp-client: ipv4 adress should not be marked ipv6 even if socket is ipv6

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

rtsp-sdp: Parse framerate caps field and set SDP attribute

The SDP attribute and its format is described in RFC4566.

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

rtsp-sdp: Parse width/height from caps and set SDP attribute

The SDP attribute and its format is described in RFC6064.

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

rtsp-sdp: add bandwidth line

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

Automatic update of common submodule

From 5edcd85 to 098c0d7

tests: add dynamic payloader prepare/unprepare check

media: release lock when removing fakesink

stream: set elements to NULL before removing

When removing a stream, set the elements to NULL first. This avoids
element-is-not-in-NULL-state errors when we dispose the elements.

Automatic update of common submodule

From 3cb3d3c to 5edcd85

media: listen to pad-removed signals

Listen to the pad-removed signal and remove the stream associated with the
removed pad.
Add signal to be notified of the removed pad.
Remove the fakesink in unprepare()
Fix signatures of the signal methods

tests: add example of reusable pipelines

stream: add method to get the srcpad

check: add media prepare/unprepare test

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

media: disconnect from signal handlers in unprepare()

We connected to the pad-added and no-more-pads signals in prepare() so
we need to disconnect from them in unprepare().

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

media: don't free streams array

Don't free the streams array in the unprepare() method, they were not
added in prepare().

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

media: don't unref the pipeline in unprepare

Unprepare() should undo what prepare() does. Because the pipeline is
not created in prepare(), we should not unref it in unprepare()

stream: clear session and caps for reuse

Set the session and caps to NULL after unref otherwise we might unref
them again later.

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

client: send out teardown signal before tearing down

The advantage is that in the signal handler you get direct access to
information about what streams are about to get torn down (in the
GstRTSPClientState).

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

client: expose connection

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

Automatic update of common submodule

From aed87ae to 3cb3d3c

media: add method to get the base_time of the pipeline

Together with a shared clock, this base-time could eventually be sent to
the client so that it can reconstruct the exact running-time of the clock
on the server.

media: add GstNetTimeProvider support

Add a property to let the media provide a GstNetTimeProvider for its clock.
Make methods to get the clock and nettimeprovider
Add a x-gst-clock property to the SDP with the IP and port number of the nettime
provider and also the current time of the clock. This should make it possible
for (GStreamer) clients to slave their clock to the server clock.

Automatic update of common submodule

From 04c7a1e to aed87ae

media: wait for buffering to complete

Wait for buffering to complete before changing the state to the target state.

media: small cleanup

tests: remove extra unref in test_setup_non_existing_stream

The unref is not needed anymore, teardown runs without it.

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

tests: GSocketService cleanup in test_bind_already_in_use

Use g_socket_service_stop so the rtspserver test stops listening for
incoming connections in test_bind_already_in_use.

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

rtsp-media-factory: g_signal_connect_object is not thread safe, can't use it here

Instead use a GWeakRef which is safe to use

This is a known GLib bug, see:
https://bugzilla.gnome.org/show_bug.cgi?id=667145

rtsp-media/client: Reply to PLAY request with same type of Range

Remember the type of Range from the PLAY request and use the same type for
the reply.

rtsp-client: expose uri

tests: Hold ref while creating second media

To test if the media aren't shared, make sure we keep the first one while creating a second
otherwise the same memory address may be reused.

configure: remove out-of-date comment

.gitignore: ignore more build files

tests: use right _LIBS variable for gst-plugins-base libs

check: add librtp to libs

tests: Add test to check selecting a port the server will send from

tests: Make sure packets are actually received

stream: Select unicast address from pool if appropriate

stream: Properties are always there in Gst 1.0

tests: Add tests for unicast addresses in pool

address-pool: Verify that multicast addresses are used for multicast and vice-versa

address-pool: Add unicast addresses

rtsp-server: Limit the number of threads per server instance

If we exceed the maximum, just round robin the clients over the existing
threads.

rtsp-server: No need to store the GMainContext in the client context

tests: Add test for client disconnection

tests: Test client and session timeouts with multiple threads

Document locking and its order

tests: Test that slow DESCRIBE don't block other clients

tests: Add tests for client-requested multicast address

docs: Put the various functions in the right sections