Fix regression introduced by https://gitlab.freedesktop.org/libnice/libnice/commit/da41258a21102f63ec5d5b2dc20d303f772eb195. Fix #64.

conncheck: Don't lookup prflx pair for UDP candidates

Code comment mentions peer reflexive candidate pair should be selected
only in the case of TCP local candidates, but it was done in all cases.

meson: Fix build with MSVC and non-English locale

Otherwise, MSVC will complain and cause build error.

Deploy documentation

Version 0.1.16.1

Version 0.1.16

gst: Fix URI in plugin

agent: Replace closed signal with GASyncResult type function

This makes the API more GIO style

agent: add nice_agent_close_async() API

Asynchronously closes resources the agent has allocated on remote
servers.

agent: Add "closed" signal

discovery: Asynchronous removal of TURN refreshes

When deleting a turn refresh, ensure the assocaited port allocations on
the TURN server are properly removed. This means wait for a response for
our 0-lifetime refresh request and implement retransmissions. Only
delete the refresh after TURN port removal has been confirmed or all
our retransmissions timed out.

component: Refactor nice_component_clean_turn_servers()

First collect the relay candidates, then dispose them in a loop.

discovery: Remove unused refresh_prune_socket()

agent: G_GNUC_FALLTHROUGH macro for compatibility with older glib

conncheck: NOMINATION STUN attribute support

Attribute is proposed here: https://tools.ietf.org/html/draft-thatcher-ice-renomination-00
WebRTC supports this attribute: controlling side provides attribute value increased by one each time selected pair is changed:
https://chromium.googlesource.com/external/webrtc/+/3c7d599750405bc734e1d5adbf1b54265b725a9d/p2p/base/p2ptransportchannel.cc#1821

Added OPENSSL_LDFLAGS

Replaced OPENSSL_CFLAGS with OPENSSL_INCLUDES

component: Fix use-after-free

conn_check_prune_socket() was trying to access deleted NiceCandidate
instance. Prune the socket before removing the candidate.

Closes #73

test-new-trickle: remove superfluous expression

tests: enable "trickle-ice" mode

Call nice_agent_peer_candidate_gathering_done () where appropriate.

test-new-trickle: prevent deadlock in dgram_process()

The STUN thread may be blocked waiting for stun_signal_ptr so signal it
when stopping the thread.

tests: rename test-dribble to test-trickle

agent: trickle ICE mode

In this mode, activated by setting "ice-trickle" property of a
NiceAgent to TRUE, a component without a successful connectivity check
will wait indefinitely for more remote candidates to come.

nice_agent_peer_candidate_gathering_done() must be called after
receiving the last remote candidate for the component to finally change
its state to NICE_COMPONENT_STATE_FAILED.

win32: remove obsolete vs9 build files

Windows users should just use Meson for building libnice.

libnice.sym: export nice_agent_new_full

Forgotten to add in 0.1.15 release?

test-send-recv: Fix occasional timeouts

While waiting for write thread to finish sending, keep also receiving so
that any STUN messages from the peer still get processed.

stun: Fix LibreSSL support

Closes #71

agent: don't print TURN password unless the logging is verbose

Limits accidental leakage of secret information.

stunbdc: initialize Winsock API when on Windows

turn: Allow lifetimes under 60s

Don't set a timeout to the very far future if the lifetime sent  by the
server is under 60s. Instead of doing "lifetime - 60", switch to lifetime/2.

component_io_cb: Avoid potential deadlock

Don't unref NiceAgent while holding agent mutex. If it's the last
reference, nice_agent_dispose() will try to acquire the mutex again and
deadlock.

test-fullmode: Make test non-racy too

test-drop-invalid: Fix raciness

gitlab-ci: Move docker images to main libnice repo

meson: Require very latest 0.49.1 as older versions break in our CI

Also remove workarounds for the older versions.

gitlab-ci: Work around meson bug for non-POSIX utf-8 characters

gitlab-ci: remove username for now because meson craps out on accents

gitlab ci: Use new CentOS 7 dockerfile based containers everywhere

tests: Add Dockerfiles

turn: Allow alternate server error with no valid credentials

coturn doesn't authenticate us before sending us to an alternate server.

stunagent: Replace more magic numbers with defines

turn: Replace magic numbers with define

gitlab-ci: Add gtkdoc-check log to build artifacts

meson: install .pc file for static gstreamer plugin

gitlab-ci: Use compiler warning level 2 in Meson build

tests: Fix -Wcast-function-type warnings with GCC 8

glib documentation says G_SOURCE_FUNC() macro should be used to suppress
the warning about incompatible function types when the function will be
cast back to the correct type before it is called by the GSource.

tests: Disable UPnP

An application using more than one NiceAgent instance may crash due to
a race in gUPnP.

Fix is to be released in gUPnP 1.1.2 / 1.0.4:

https://gitlab.gnome.org/GNOME/gupnp/commit/0123e574595e0a547ce26422633df72d63d3d0e0

meson: unify exported symbol versions with autotools

libnice.so created by autotools has its symbols bound to the default
'Base' version node, whereas Meson build arbitrarily introduced a
version node named 'libnice'. This breaks tools like dpkg-gensymbols
that track changes in exported symbols through several releases of the
library.

Remove 'libnice' version and revert back to using unspecified version.

agent: Fix spelling

Errors reported by lintian -EvIL -pedantic.

udp-turn: Avoid potential integer overflow

udp-turn: Fix unaligned memory access on ARM

gitlab-ci: Build gtk-doc and test install with meson

gitlab-ci: Split make distcheck into a separate build

gitlab-ci: Run autotools tests in CI too

gitlab-ci: Add meson tests to build

gitlab-ci: Skip meson build on schedules

test-tcp: Use IPv4 instead of IPv6

This is because Docker doesn't have IPv6 by default.

tests/test-different-number-streams: Make it work with only localhost

meson: fix test-gstreamer

On Nix, test-gstreamer fails with

36/38 test-gstreamer                          FAIL     0.07 s (exit status 1)

--- command ---
BUILT_WITH_MESON='1' /build/libnice/build/tests/nice-test-gstreamer
--- stdout ---
Running suite(s): nice_gstreamer_test
0%: Checks: 1, Failures: 1, Errors: 0
../libs/gst/check/gstcheck.c:466:F:nice:buffer_list_test:0: Could not create a 'nicesink' element
Check suite udpsink ran in 0.002s (tests failed: 1)
-------

Let’s add the build directory to GST_PLUGIN_SYSTEM_PATH_1_0, so that
the test can find the built GStreamer element.

meson: Fix DLL search path during test execution on Windows

Meson needs to see all libraries a target depends on so that it can add
each to PATH when running "meson test".

Avoids "*.dll was not found" errors.

meson: Add fallback for openssl

Version 0.1.15.1

Version 0.1.15

Makefiles: Add meson.build files to tarball

Add more files to .gitignore

Fix little bugs found by clang-analyzer

docs: Add _get_type() symbols to hidden section

meson.build: Sync libversion with libtool

nice/test-symbols.sh: Make symbols test work from other path too

Makefile.am: Export *_get_type() symbols like meson does

conncheck.c: Fix negotiation problem

When a peer reflexive candidate is discovered, it is added to the
list of connection checks, but is is not nominated by default.

This causes a problem if parent pair is already nominated and
the remote peer does not send more requests because it has already
selected a candidate. (i.e.: This is the case of WebRtc in Firefox)

agent: Check for early checks on dribbled candidates too

We have to check this list every time candidates are added.

conncheck: Don't free all incoming checks on credentials

Candidates could also be dribbled.

component: Make incoming_checks into a GQueue

We check the size regularly, so add a little efficiency.

meson: Allow building of g-i with static libs.

Allow building of g-i with static libs which should
work since Meson 0.49 and sufficiently recent
gobject-introspection (>= 1.58.1). If g-i is too
old people will get a suitable error message and
can disable g-i via build options.

meson: Fix pkg-config generation with internal dependencies

Prevents Meson error:

ERROR:  requires argument not a string, library with pkgconfig-generated
file or pkgconfig-dependency object, got <InternalDependency null: True>

Requires Meson 0.49.

meson: Fix build with fallback glib

When building libnice with glib fallback, its 'libgio_dep' internal
dependency doesn't provide all needed include and library search paths.

To avoid unresolved header files and linker errors in such case,
'libglib_dep', 'libmodule_dep' and 'libgobject_dep' from glib submodule
must be pulled in as well.

The problem should be fixed in GLib 2.60.

agent: Fix use of freed stream in nice_agent_dispose()

nice_stream_close() calls agent_find_stream(), which iterates over
agent->streams list.

Therefore, when we're freeing agent->streams in a loop, we must also
delete the respective GSList item from the list at the end of each
iteration. Otherwise agent_find_stream() would stumble over already
deleted NiceStream instance the next time it gets called.

Fixes random crashes when running test-different-number-streams on
Windows.

meson ci: Add logs to artifacts

pseudotcp: Rename state enums to not clash with system ones

tests: Initialize Windows Sockets where needed

Fixes failing test cases on Windows.

udp-bsd: Fix recv with small buffer on Windows

The underlying GSocket implementation on Windows returns an error when
the user-provided buffer isn't large enough to fit the whole datagram
received on a message-oriented socket.

When this occurs, in order to preserve identical behavior of udp-bsd
NiceSocket across platforms, we have to mute the error and set the
received message length to the size of the provided NiceInputMessage.
Any excess portion of the message gets discarded.

Fixed udp-bsd test on Windows.

GLib 2.48 is required in order to use G_IO_ERROR_MESSAGE_TOO_LARGE.

agent: Expose input_message_get_size() in agent-priv.h

libnice.sym: add some generated get_type functions and fix gtk-doc build

https://gitlab.freedesktop.org/gstreamer/gst-build/issues/14

meson: tests: make fullmode-with-stun non-parallel and disable fuzzing test

test-fullmode-with-stun needs to run in non-parallel mode to work.

test-pseudotcp-fuzzy sometimes just times out, which needs investigating.

meson test -C builddir --repeat=20

windows: use proper fallback for ssize_t and use stdint.h for types

stdint.h is available since vs2010 and stdbool.h since vs2013,
so we should be able to just use that these days.

Fix build with MSVC

Tested inside Cerbero with VS 2017

meson: fix gtk-doc message

It's disabled by default, so saying it wasn't found is confusing.

stun: tools: drop getopt for arg parsing for better windows compatibility

Just implement argument parsing ourselves here.

It's not really more lines of code than the getopt.h variant
and makes things build on Windows with MSVC without having to
ship a separate getopt implementation.

CI: rename autotools build target

No tests are being built or run here.

CI: use -werror for meson

And drop gstreamer1-plugins-base-devel dep
which is not needed.

CI: Add a meson build job

meson: disable introspection if g-ir-scanner is not found

meson: add fallbacks for glib and gio

meson: don't error out if clock_gettime() was not found

It's not a requirement, there's a fallback.

meson: check for gtkdoc-scan

meson: add new unit test

meson: allow selecting crypto library manually via an option

Add support for Meson build system

agent: Don't require stream names to parse remote SDP

The streams are matched by the order in which they appear in the SDP,
not by name, so insisting on local streams to have one makes no sense.

This likely should have been removed in 5972db0.