GitLab

This reverts commit fa77bda4.
and commit 5a9d1043

We got a sever regression on OpenDHT since storage feature introduction.
We revert to a known good state.

Change-Id: Id2af93e68b3ec943db34f223b1391cfe6fc9deb5

Adrien Béraud authored Dec 10, 2015 and Guillaume Roguez committed Dec 10, 2015

This bump tries to import opendht changes to fix an network overloading
situation.

Tuleap: #104
Change-Id: Ic567fbee927e7acd22c71075c3546dab52722819

Tuleap: #104
Change-Id: I03bdfaeae256e47c49adf5800c85ee6ad7c9e7f9

Adrien Béraud authored Nov 27, 2015 and Guillaume Roguez committed Dec 09, 2015

The "OVER" event is the point in time from which the daemon won't know
about this call ID anymore.
There is always exactly one OVER event when a call is removed.

Tuleap: #102
Change-Id: Id11b9047b27c33258b731f7843fd7241e4c7c07f

This commit adds type verification when an AVCaptureDevice is added.
The notification that a new device got connected, is received for both
video and audio devices.

Change-Id: Id48fb4182589234df164ab6337fb5d4863987a15
Tuleap: #149

New developments in libav allow size selection.
This patch adds this ability in our OSX video device implementation.

Tuleap: #166
Change-Id: Ibe9c38e85b5ce2a0bee4b0528390aae716a1aee7

WARNING: API changes

This patch makes code common for IM send/receive message API's
between call and account classes.

Multi-part messages is supported for :
- SIPAccount
- SIPCall (any SIP like account)

RingAccount supports only Single-part messages yet.

Change-Id: Ic65425cd95f02f89f03dd6ea2a9c4a10bb233859
Tuleap: #157

Eloi Bail authored Nov 30, 2015 and Eloi Bail committed Dec 03, 2015

When several calls were performed, quality and bitrates of previous
call were used. When stopping VideoRtpSession, we make sure to reset default
values.

Change-Id: Ibe7b58f33979fae734dbbe3f7c31bd4d57c5a13f
Tuleap: #156

Eloi Bail authored Dec 03, 2015 and Guillaume Roguez committed Dec 03, 2015

If a keyframe request is never answered, a timeout is reached
and the call failed.
As keyframe requests are transmit over UDP, they are too easily dropped,
on connection like wifi.

This feature is not mandatory for communication and until we found
a better solution this patch disables the request.

Change-Id: I3d5e8d0e3d0119db52fbfc0f37ce2949224b1806
Tuleap: #167

Julien Grossholtz authored Nov 05, 2015 and Guillaume Roguez committed Nov 25, 2015

Ring daemon clients need to know what data is streamed to a client
during a call. This patch creates an VIDEO_SOURCE field within getDetails
that contains a string with thei video source used by the daemon.
In case there is no video the string will be empty

Change-Id: I4e5c8a1dcdbd375c327bd20c6674e9b2c2d62bf5
Tuleap: #99

This commit adds a patch from libav resolving issues with h263p encoder.
see https://bugzilla.libav.org/show_bug.cgi?id=912

Change-Id: I95f5e81063af01a57be47b70fadaf54c3393eb52
Tuleap: #138

Corruption was seen on transfert. Hangup of a call was done while
the call was destroyed. Use of shared_ptr resolved it.

Change-Id: I11b2b91e07bc1da77f05302fa08926193923af60
Tuleap: #122

- TCHAR conversion to std::string wasn't working

Change-Id: I8fa9a634f50a58515a0ba49be99dd166a0a537b3
Tuleap: #133

Eloi Bail authored Nov 24, 2015 and Guillaume Roguez committed Nov 24, 2015

Attended transfers led to an assertion in pjsip. According to high-level
librairie pjsua, unused in Ring, and specially function pjsua_call_on_incoming,
it looks like we should not call pjsip_inv_answer when replace header is
present. This function actually leads to the pjsip assertion.
pjsip_inv_initial_answer should be rather used.

NOTIFY is still missing.

Change-Id: I2e4a09533bd33dd32b2cc61a7d9737dc1fe79738
Tuleap: #122

Current size was defined before the LibAV jitter buffer
was introduced. The buffer size defined was already very
conservative: 32000 samples means 2/3 of a second (an eternity).

* reduce buffer size to 16000 samples (333 ms at 48kHz)
* reduces audio latency of up to 333 ms
* should slightly improve echo cancelling

Tuleap: #120
Change-Id: I0c3d4f83d18f4fdede7e89cd9a36327af716c1ca

8.36 is not hosted anymore. Bump to latest 8.38.

Tuleap: #124
Change-Id: Ib40e60979d2ac1909cf8810a45a7dc9a0bf013cc

Eloi Bail authored Nov 19, 2015 and Guillaume Roguez committed Nov 19, 2015

Padding needs sometimes to be done for scaling purpose when a change of video
source is performed. Padding calcul is wrong and leads to memory bad writting
access.

Tuleap: #87
Change-Id: Ied282941077c1e268529457d39cdcc403a741aaf

Eloi Bail authored Apr 28, 2015 and Guillaume Roguez committed Nov 18, 2015

H263 version 1 allows a limited range of resolutions outdated compared
to recent camera capabilities. It was thus nearly impossible
to use it on Ring.

This patchset forces encoding of H263+ which supports all resolutions
multiple of 4 from 4x4 to 2048x1152.
On Libav we see video artifacts if multithread is enabled
on encoder side. A workaround deals with forcing h263 encoder
to monothread.

See https://bugzilla.libav.org/show_bug.cgi?id=912

Tuleap: #138
Change-Id: I7121e182fa13139d28db010e2ed0de573c616de7

IceTransport::isXXX() API are not thread-safe, not clear in which
state the transport is and not easy to manipulate (order of state).

This patch tries to solve that by:
- Procted API by a mutex
- Remove uneeded isComplete() API
- Ensure that each API are ordered (one at true implies true
  on all lower states: INITIALIZED -> STARTED -> RUNNING).
  All return false on Fail state.
- Change usage accordingly

Tuleap: #107
Change-Id: I17211e54322d70bbfe18c28f06cf9967b9ef93d2

Adrien Béraud authored Nov 11, 2015 and Guillaume Roguez committed Nov 16, 2015

Ant 1.9.5 is not hosted anymore, which prevents Android to compile.
Bump to 1.9.6.

Tuleap: #62
Change-Id: Ib20a0b6ea40d229ddfd09e7850b4606d9f28f508

Eloi Bail authored Oct 28, 2015 and Guillaume Roguez committed Nov 13, 2015

Dynamic quality change was done by increase/decreasing encoder bitrate.
It led to video artefact issues, specialy at low bitrates.
In addition, it does not make sence to use same bitrate with different
resolutions and fps. With high resolutions and fps, maximal bitrate
need to be higher.

It is possible for H264 and VP8 to set video quality using CRF.
This parameter solves video artefact issues and choose a different
output bitrate depending on input resolution and fps.

Tuleap: #80
Change-Id: I07d37175dfc335b5fc70cf54798f82eafe921ea9

Adrien Béraud authored Nov 04, 2015 and Guillaume Roguez committed Nov 12, 2015

This patch removes annoying build warnings on non casted unions.

Tuleap: #13
Change-Id: I9ec70dc559dea5973539b655b4d7a24ab22351ef

Make sure to not include OpenSSL when GnuTLS is used.

Tuleap: #77
Change-Id: I6f3ee4b80471d2562c6c5ef1ebb17cdc0002760a

Edric Milaret authored Nov 11, 2015 and Eloi Bail committed Nov 12, 2015

- dshow provide min/max value that don't always work. No pattern
have been observed as to why...
- ffmpeg chooses the highest value it can when we don't provide it.

Change-Id: I8d8b0993ded6d474f3ad9c741df7fd14fb61021b
Tuleap: #132

See http://trac.pjsip.org/repos/milestone/release-2.4.5 and similar for
detailed changelog.

* Bump from 2.4 to 2.4.5
* Mostly bug fixes
* Should fix #77
* Removes the need for patch "mainstream_fix_1844"

Tuleap: #77
Change-Id: Ib457203082fd7326d05bc8910a2aa2567974a4b8

Tuleap: #13
Change-Id: Ic948160e96a0f5e232871863407fa920292318ac

Let user choose the RING_REVISION and RING_DIRTY_REPO values.
This also permit packaging to build from non-git sources tree
with a fancy and valid revision string.

Just set previous environment variable before to run
the make command.

Change-Id: I24d71f985decc2187f09ab21eb72e588ede7fb7a
Tuleap: #71

Eloi Bail authored Nov 05, 2015 and Guillaume Roguez committed Nov 06, 2015

AVPacket are not freed after use. The solution deals with
allocating AVPacket on the stack and ensuring to free it
calling av_packet_unref.
This fix has also widly deployed on Video for having a unique
method other the code.

Change-Id: I049b29effb4e6748ee2dd12b4c52536c99fef498
Tuleap: #91

* use previous gettext version 0.18.3
* should fix build on Ubuntu Trusty 14.04
* keep gettext 0.19 on windows for mingw compilation

Tuleap: #62
Change-Id: I39f3d5b4938f7b2926f730a3b7822c19e05d9d1e

* bump from 3.4.2 to 3.4.6
* mostly fixes (including security)

Tuleap: #62
Change-Id: Ibe0e44e91f5cc7d9bb102aa9af9fed03c65f5754

This patch fixes a memory leak in SipsIceTransport class.
gnutls_priority_init() allocates memory, no gnutls_priority_deinit()
is called to free it.
This last has been added in SipsIceTransport destructor.
This patch also fixes code rules error on priority_cache_ naming
(this is the variable initialized by gnutls_priority_init).

Change-Id: I32a8a7347d4af96886a0def2d9a6d77fe0fad7c7
Tuleap: #92

Issue: #77219
Change-Id: I5bf71251212343e71f5e04081615a5890f7aa345

bump:
* cmake to 3.2.2
* libtool to 2.4.6
* automake to 1.15
* m4 to 1.4.17
* tar to 1.28
* xz to 5.2.2
* gettext to 0.19.6

Tuleap: #62
Change-Id: I7fa1f021ab1c8425d05416dc652aad060b2c2536

Adrien Béraud authored Oct 26, 2015 and Guillaume Roguez committed Oct 28, 2015

* add "active" runtime property to accounts
* accounts can only be used when active
* "active" state is not persistans
* add "active" state to volatile properties map
* add setAccountActive API
* "active" is the default state (backward compatibility)

The active state can be used to manage runtime states
defining if an account should be used (e.g. on Wifi/3G etc.)

Tuleap: #9
Tuleap: #42
Change-Id: Ie3b3aaadea6e43237fc22f6a1f96de8cbfe2bd95

Alexandre Lision authored Oct 22, 2015 and Guillaume Roguez committed Oct 27, 2015

missing override keyword to tag overridden virtual methods.

Tuleap: #69
Change-Id: I27d10cc49835e0e42858912cdfa11ef3e8761c04

* bump MsgPack to stable release 1.2.0
* bump OpenDHT to latest master supporting MsgPack 1.2.0
* adapt RingAccount to latest OpenDHT API

Issue: #81946
Change-Id: I331626120ca5e1c4c0d6f9a2dcf69e8d2934a636

Adrien Béraud authored Oct 15, 2015 and Guillaume Roguez committed Oct 23, 2015

Tuleap: #3
Change-Id: If43ac48729f4f55ba7fa02faa548b69d7f41dc78

Adrien Béraud authored Oct 13, 2015 and Guillaume Roguez committed Oct 22, 2015

Tuleap: #13
Change-Id: I8bc2d5e5b73bfb9461aeaea13ec1923567c64e7d

Calling this API causes a race condition with RtpSession::start().
The former needs the latter to be run first, this was not enforced,
causing a crash by nullptr access on socketPair_ member.
This patch checks for a non-nullptr socketPair_ to procceed the restart.

Issue: #81987
Change-Id: I20886fe70398f018621aade1c7bac3ff15ef9055

Call::getCallId() method is accessed in a nullptr pointer context.
This patch fix it by using the user given callId.

Issue: #82045
Change-Id: I8700aeaa173706909cf303be6d1efff5a4d1fc44