Do not retransmit triggered checks if reliable transport

Small refactor for conncheck add_pair function

Print info on selected pair

Store connections in tcp-passive to allow sending

In the case of TCP-ACTIVE, we don't have a problem, because the new
socket will generate a  peer-reflexive candidate with the right
sockptr, and all sends will go through it, but for TCP-PASSIVE, there
is no peer reflexive candidates, so once the pair is selected, the local
candidate will point to the TCP-PASSIVE candidate with the TCP_PASSIVE
socket type which will cause all sends to fail, so we need to proxy
them to the child connections

Fix RFC4571 n_buffers when reading and frame outgoing STUN messages

Fix SDP generation for TCP candidates

Do not create a socket_source if fileno is NULL

Add support for RFC4571 framing (packetized mode) for ICE-TCP candidates

Disable reliable mode for test-fullmode, we already have pseudotcp unit tests

Add RFC4571 framing to ice-tcp for MS-TURN

Data and STUN messages sent over TCP MUST follow the framing method
specified in [RFC4571] section 2. We will frame every outgoing packet
and split it into 62KB frames when sending, and on reception, we will
unframe the rfc4571. For now, we only unframe for udp-turn tcp-act/pass
candidates, which means only for MS-TURN over TCP.
We still need to do the unframing for real ice-tcp candidates, then add
support for reading the data in a serialized way.

Set pair->sockptr when discovering peer-reflexive candidates

Fixes Issue #1

Add tcp-passive and tcp-active socket layers with unit test

Add a nice_socket_recv convenience function similar to nice_socket_send

Refactor tcp-bsd to ease integration of tcp-act/tcp-passive

Make socket reliable in case of tcp-act/tcp-pass and move logic for ms-turn

Factor out the add_to_be_sent into a helper API for sockets

The add_to_be_sent was duplicated in http/socks5/pseudossl/tcp-bsd and
had some small differences. It's better to factor it out so bug fixes
get applied to all of them and code is cleaner.

Do not allow message_offset to become negative when queuing messages

Use reliable TCP for ms-turn since it generates tcp candidates

Do not return the socket to the app if we use ice-tcp, even in unreliable case

Add support for MS-ICE2 candidate priorization for OC2007R2 compatibility

Rename turn files and API to be UDP specific

The TURN-TCP RFC is very different from TURN-UDP, the current turn and tcp-turn
files were only for UDP TURN RFC and UDP TURN using TCP transport.
When we implement TCP-TURN, we'll need a different algorithm, so I
renamed the files and API to avoid any confusion.

Fix crash when discovering peer reflexive candidates in RFC compat

test-fullmode: protect against ready->connected->ready state change and accept data reception on connected state

Make OC2007 relay candidates use TCP_ACTIVE and TCP_PASSIVE candidate types when using TCP TURN

Do not add server-reflexive from TURN if using a TCP connection

Remove priority functions from public header file and move to agent-priv.h

Change priority algorithm for better values

Now we will always give a handicap to UDP candidates, but still give
relays the lower priority. We have the following priorities now :
In reliable mode :
TCP Host : 120
TCP peer-refl : 110
TCP nat-assist : 105
TCP srv-refl : 100
UDP Host : 60
UDP peer-refl : 55
UDP nat-assist : 52
UDP srv-refl : 50
TCP relay : 10
UDP relay : 5

In unreliable mode :
UDP Host : 120
UDP peer-refl : 110
UDP nat-assist : 105
UDP srv-refl : 100
TCP Host : 60
TCP peer-refl : 55
TCP nat-assist : 52
TCP srv-refl : 50
UDP relay : 10
TCP relay : 5

test-priority: Unit test various configurations of candidate priorities

Fix peer reflexive priority depending on transport and compatibility

Add support for ice-tcp priorities, udp-tunneled and nat-assisted priorities

Use nice_candidate_ice_priority instead of _full variant when possible

Add support for MS-TURN in OC2007 compatibility turn usage

Do not enable keepalives for tcp candidates

do not set socket tos if fileno == NULL

Add support for tcp candidates in sdp generation

Add candidate type to debug

Add a sockptr to CandidateCheckPair, will be required for tcp-active

Do not attach to sockets with fileno == NULL (requires for tcp-active)

Only check selected pair for writability in gio outputstream

Do not retransmit connection checks on reliable transport

Use local candidate foundation as CANDIDATE-IDENTIFIER when creating
STUN_BINDING requests.

Candidate check pair foundation was used - a combination of local
and remote candidates' foundation, which does not correspond with
[MS-ICE2] 2.2.2.1 0x8054 CANDIDATE-IDENTIFIER

Disable TURN refresh in MSOC compatibility mode

At least temporary, maybe it is not used at all. TURN refresh messages
were not seen in the packet capture.

If allow_partial, then allow a return 0 from the socket

Once a ice-tcp connection is established, it should work like any other, so just send to it

Generate valid TCP connection check pairs

Refactor discovery/agent to add support for non-udp candidates

Fix indentation for _set_remote_candidates_locked

Small refactoring of agent, loop on component id from 1

Add support for MS-TURN in tcp-turn

Add support for MS Office Communicator pseudossl compatibility

Factor out pseudotcp so it's not used if the transport is already reliable

Add a nice_socket_send_reliable API for internal use.

One issue with tcp-bsd is that it will queue messages when the tcp socket
is not writable, but it will also drop messages when the queue is full.
If we want to do proper reliable ice-tcp, we need to make sure that
messages don't get dropped, also, this would affect http/socks5/pseudossl
if their messages get dropped. For ice-tcp, when the socket is not writable
we want to return 0, not queue the message.
The change here is to allow connchecks and other important protocol messages
to be sent as 'reliable' on tcp transports by queuing them and in the various
socket layers and to never drop them, but all user messages will be dropped.

if the tcp socket is not writable.

turn: In case of error, don't just pass through the data, return an error

Return number of sent messages in socket_send_messages when an error happens part-way

Fix return value of send_messages in socks5

Allow tcp-bsd to act as reliable or non reliable transport and fix is_reliable on other sockets

Add a socket type for socket layer

Some refactoring to allow for ice-tcp (non pseudotcp reliable)

Fix documentation in preparation for ice-tcp support

Add ICE-TCP candidate transport types

Small documentation fix

agent: Use 1280 instead of 65536 buffer size to send STUN Message

RFC 5389 says:
   All STUN messages sent over UDP SHOULD be less than the path MTU, if
   known.  If the path MTU is unknown, messages SHOULD be the smaller of
   576 bytes and the first-hop MTU for IPv4 [RFC1122] and 1280 bytes for
   IPv6 [RFC2460].

So sending 65536 bytes is always wrong

Version 0.1.7.1

Version 0.1.7

stunmessage: Revert another function to use sockaddr

The farstream unit tests were using it.

threaded-example: Make sure the foundation string is NULL terminated

Version 0.1.6.1

Version 0.1.6

Fix udp-turn returning wrong message length in socket_recv_messages.

When using optimal path where recv message has only one buffer, the
buffer length was set to the size of the buffer, not its length, which was
causing the wrong message length to be returned causing the agent to
discard STUN messages because their size doesn't match received data
length.

agent: Add a missing forward declaration

agent: Simplify gathering-done stage of UPnP handling

This introduces no functional changes, but consolidates and documents
the code a bit more.

agent: Tidy up upnp_mapping handling

Use a NULL check to determine if upnp_mapping is an empty list (rather
than counting every element). Use g_slist_free_full() where appropriate.

This introduces no functional changes.

agent: Document units for NiceAgent::upnp-timeout property

It’s in milliseconds.

stunagent: Only declare the long term key valid if the username/realm coudl be read

example: Prevent possible non-NULL terminated string

address: Correctly identify ipv6 link local

The test for fe80::/10 always returned FALSE.

add nice_agent_forget_relays()

component: Clear turn local candidates when clearing turn servers

But keep the turn connection that's currently being used if it is
the selected candidate. Also clear the TURN candidate refresh.

agent: Make the TurnServer structure refcounted

Instead of just blindling assuming that all of the pointers are valid
until the end.

discovery: Use g_slist_free_full for candidate refresh list

discovery: Use g_slist_free_full for discovery list

discovery: Remove never used and leaked msn_turn_* fields in CandidateDiscovery

discovery: Remove never used and leaked msn_turn_* fields in CandidateRefresh

agent: Factor out creating the TurnServer structure

discovery: Remove duplicated element between CandidateRefresh and NiceCandidate

discovery: Store NiceCandidate in CandidateRefresh

component: Make detach_source close the socket

component: Make is possible to remove a socket

Change the SocketSource handling to make it possible
to remove a source, not only to add one.

factor out: component_clean_turn_servers()

stream: No need to pass RNG to stream_restart()

It can now take it from the agent.

conncheck: Only print debug message if actually freeing something

conncheck: Don't fail a stream that has on-going discovery

agent: Add nice_agent_restart_stream() to restart a single stream

stream: Clear connchecks on restart inside stream

conncheck: Use g_slist_free_full() for conncheck list

Skips one iterator over g_slist_foreach() + g_slist_free(). It is also easier
to read.

conncheck: Make conn_check_free_item() static

It is never used outside of this file.

conncheck: Only stop conncheck timer if no stream has pending conn checks

conncheck: Remove useless return value on conn_check_prune_stream

conncheck: Factor out stopping connectivity checks

component: Remove return value from *_restart() functions, they can't fail