Aline Gondim Santos authored 4 years ago and Sébastien Blin committed 4 years ago

GitLab: #422
Change-Id: Id94eb245d9675d305a0d05da76e62bcd245525ec

- adds a missing space in the configure command line
- removes some python3 warnings

Change-Id: I79458a408a6f3198cb329072abf530ed8f23986a

Change-Id: Iff69c5888acba0a9a165bce4f4106a217dc78134

This allows the rendez-vous to be considered as a
conference and not as participant for the other
conference.

Change-Id: I9053533a605f1928da93ed3d3e9235db8b764591

Audio sender doesn't need audioInput, audio rtp session already has one.
(Consistency with video rtp session)
muteState is not used in audio sender.

Change-Id: I7c798c94050fa6533ff9b726914f2f880e426a52

Pierre Lespagnol authored 4 years ago and Sébastien Blin committed 4 years ago

Gitlab: #453

Change-Id: I9a870b2f930c69fe26432f1f15169d13365e702d

Sébastien Blin authored 4 years ago and Adrien Béraud committed 4 years ago

On some very slow network, the process to connect to a relay
candidates can take a lot of time. Cause this timer includes
the time between the ConnectionAttempt and sending the
Binding response. With some packet loss it can quickly increase
this delay.

Example:
Classic nego (Successful ICE):

10:36:28.126 .RX 144 bytes STUN message from 51.222.10.40:3478:
--- begin STUN message ---
STUN ConnectionAttempt indication (...)

10:36:29.378 .TX 84 bytes STUN message to IP:45320:
--- begin STUN message ---
STUN Binding success response

Problematic case on a bad network:

// Bad network peer
09:39:10.910 .RX 288 bytes STUN message from 192.252.140.236:3478:
--- begin STUN message ---
STUN ConnectionAttempt indication

// other PEER:
09:38:51.855 192.252.140.236:11846 (nominated): connectivity check FAILED:
STUN transaction has timed out (PJNATH_ESTUNTIMEDOUT)
=> The other peer close the socket

// Bad network peer
09:38:52.258 .RX 76 bytes STUN message from IP:36436:
--- begin STUN message ---
STUN ConnectionBind success response

9:38:53.756   tcprel0x7f7098178460  .socket send(): Not found (PJ_ENOTFOUND)

The value of 15 secs is chosen to follow ice_inactive_timeout in
the webrtc's sources (CONNECTION_WRITE_TIMEOUT)

Change-Id: Idc66f6cb2018fbf03e34059640fa91601535dcb1
GitLab: #1201

Pierre Lespagnol authored 4 years ago and Adrien Béraud committed 4 years ago

Gitlab: #409

Change-Id: Icaf85305227bc5fdbfc2a23966c1080b7e05f0fc

Pierre Lespagnol authored 4 years ago and Adrien Béraud committed 4 years ago

Gitlab: #447

Change-Id: I249fb7034a12f117298187b7b42977760be3ef83

Rework shutdown of the UPnPContext to cleanly remove allocated
mappings. Also apply many code improvements and clean-up.
This will also prevent some crashes seen at shutdown.

Change-Id: I5cdc56df9d096d443ccfb2619edbec35bc00a6bc
Gitlab: #436

Change-Id: Ia95bd71499b502f4e6d3df690537a6e1c91422d0

Gitlab: #468

Change-Id: I2c94823bb8efab6fccddd59de1ae96f290745691

Adrien Béraud authored 4 years ago and Sébastien Blin committed 4 years ago

Change-Id: I61ff878c79b4116e28bcdce495367c73029af1e9

Change-Id: I8e98b66bf23a34cb1a4b3d360d8c1027e064c7e5

Sébastien Blin authored 4 years ago and Adrien Béraud committed 4 years ago

getting a TlsSession::shutdown() while doing a gnutls_handshake
provoke a loop (cause the handshake will return GNUTLS_E_INTERRUPTED
and will loop until HANDSHAKE_MAX_RETRY).

Change-Id: Ifabc24c815552a14781b72cb9baf66d987cb25ce

Gitlab: #456
Change-Id: Ic44c9ce0b682bf8352b15ad9eaf8d51a15c4c657

Change-Id: I139b6fcb70986be033c88bbf7e0545af326ff407

Change-Id: I49383922aca110594410b7d86aafde2ece6af4b7

Change-Id: I92a9a61c8d34736af99693ec826dd2385c319bfa

Change-Id: I5efa45a80f05c6ba4106e05234612c6ea5d6fdf9

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

This brings wget performance up to par with that of curl, which makes
downloading the contrib sources via wget about a ~6X faster.  Wget is
now also made the preferred downloader, as it's a smaller than curl,
and is GNU.

* contrib/src/main.mak (download): Remove the -p option, not suitable
for single file downloads (it's meant for retrieving all the resources
of a web page).  This fixes a warning too.  Remove the --passive
option, which is not documented.  It's probably equivalent to
--passive-ftp, which is the default anyway.  Use --waitretry instead
of -w.  The later was causing the 10 s delay to be inserted on every
HTTP redirection, wasting time.

Change-Id: Ibcd6805c66da3fb0c3f37b7128a34cf6c269f400

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

It should only worry about its parent directory, not its grandparent.
This makes it possible to bind mount just daemon/contrib in a
container, to run 'make fetch' for example.

* contrib/bootstrap: Adjust relative paths.
* contrib/src/main.mak: Express using the CURDIR variable.
(PATH): Express using the TOPSRC variable.
(pkg_static): Express using the SRC environment variable.

Change-Id: I02cba2249e4971d76ef0af0fa00f4b1af61c5aff

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

The 'fetch' target relies on the prerequisites of the .sum-$(PKG)
files.  Prior to this change, 'make fetch' would not fetch the
openbsd-libressl tarball.

Change-Id: Ic91cc7032243d0c0f3b3d69fb74cfb3c633d0fc3

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

This target can be used to print a list of the contrib tarballs
required to build the daemon.  It can be used to copy only the
required tarballs from a cache when building a source release, for
example.

Change-Id: I402c29989b3e6fed8a82418d19dbbacea8c44f50

Maxim Cournoyer authored 4 years ago and Adrien Béraud committed 4 years ago

It was previously possible by overriding the TARBALLS variable on each
make call, but that's inconvenient.  This changes allows specifying it
once, at bootstrap time.  If the TARBALLS environment is defined, it
is also honored (and takes precedence over the value provided at
bootstrap time).

* contrib/src/ffmpeg/rules.mak (ffmpeg): Because the $(TARBALLS)
variable is now made absolute (for clarity in the build output), do
not use a relative path.
* contrib/src/onnx/rules.mak (onnx): Likewise.
* contrib/src/vpx/rules.mak (libvpx): Likewise.
* contrib/src/x264/rules.mak: Likewise.

Change-Id: I07497b095938c7885159f44753ead2814ed2a2fe

Reason for revert: breaks the design

Change-Id: Ib94d36d5f671628ddfcbaaa4fef180821e7a79e3

Aline Gondim Santos authored 4 years ago and Aline Gondim Santos committed 4 years ago

Reload plugins libraries after setting or resetting preferences.
Previously unloading and reloading should be done by client.

GitLab: #418
Change-Id: Ib6938cfe63c2d6a6336adee4994da6a54be7ecd8

The default status code of 0 is passed to the done callbacks when
the requests are destroyed. Returning from the callback when we get
this code prevents locking the mutex on multiple times on the same
thread.

Change-Id: I4e6da6de7b6572d6a0f744c3b90d1be28ad0298c

Kateryna Kostiuk authored 4 years ago and Kateryna Kostiuk committed 4 years ago

Change-Id: I6cc29ad13e50cd5c58976bdc99e5deee68c3720c

Gitlab: #380

Change-Id: I3886c4c3b13710464f25ac779482e8e3726bda5c

When frame decoding is fast, av_read_frame called before new data
available. It will cause high CPU usage.
This patch adds wait time when no data is available.

Change-Id: I675092e2497f969976f797cc1bbb6b466b9dbcb5

Change-Id: Idb36ee1a0ac04c577ae454a855280d9bd63f3245

Mohamed Chibani authored 4 years ago and Adrien Béraud committed 4 years ago

Gitlab: #382

Change-Id: I9b6adba40262f2feb4bbc8445c98b52d0297a194

Sébastien Blin authored 4 years ago and Adrien Béraud committed 4 years ago

Change-Id: I6f92c33c8b0087b58e7d692bde4d3421906fb61a

Ming Rui Zhang authored 4 years ago and Adrien Béraud committed 4 years ago

Gitlab: #346

Change-Id: Ia7b37abf31a14e8648f61a2415687d2400aa9937

Change-Id: I2a59248df58ad5e019d3ded0b97e4db95c5520cb

Ming Rui Zhang authored 4 years ago and Sébastien Blin committed 4 years ago

daemon cmake ZERO_CHECK reference from ring-daemon vcxproj file
will cause the build to fail for client qt sln on windows when
using the newest Visual Studio

Change-Id: I0488f65a636b28ab1ba9a2add4550d5ce9a52ca1

4463ed9e introduced a regression
and was checking for 70000+err instead of 120000+err.

Connection refused is 120111 (PJ_ERRNO_START_SYS + 111) and
not 70111 (PJ_ERRNO_START_STATUS + 111) leading to incorrect
error detection and some negotiation failed.

This patch also introduces some of the logs I used to debug this
issue that can be useful if a similar scenario come back.

Change-Id: I2a0e23f07f07ae81db14daea5d257976463c1133
GitLab: #438

Change-Id: I6ccffb6ed28255922db2ca752503c42269b982e2

Aline Gondim Santos authored 4 years ago and Aline Gondim Santos committed 4 years ago

GitLab: #395
Change-Id: I8347ffe1a37ccf63d30e028e36e7597226d4ae66