Change-Id: I42b390cccfe5a849be4bffccb014534d0e75d723

Philippe Gorley authored 5 years ago and Sébastien Blin committed 5 years ago

Change-Id: I21af38d6a5fcd36d11955b185f0858c8bb07a278
Gitlab: #135

Change-Id: I62befaa0dbce5ac063ad117463680b202f85217c

Change-Id: I9a9b37d358b55748526720eaa9c21653bc6ab8b3

Change-Id: I96bb15b0a00e6998c32949a894698493d0e76c19

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

When always recording, the uninitialized value could break the overlay
filter, making one of the video sources not get filtered properly.

Change-Id: I137aa175f0ba5d88a660c6bcfa54f37bd65b7c9d

Some of the decoder's settings are only set once decoding has started,
such as the pixel format.

Change-Id: Ifb3cf4606d1dc02dc089bc8ca9ca11a7a3071caf

Change-Id: I96bea87e43cf08ddf0b2805f13d35c3efa6c2db7

As specified in RFC 5245, the aggressive negotiation uses the
candidates as soon as possibe. This avoid some possible latencies
due to the negotiation of an unique pair.

For our use case, this avoid a 30 seconds delay during the media
negotiation in some cases.

Change-Id: I155a0c5f5706908d5e86a08fb2d65e0e3b8c61e2
Gitlab: #140

Philippe Gorley authored 5 years ago and Philippe Gorley committed 5 years ago

MediaFrame doesn't have a ptr_ or allocated_ fields, so casting it
to VideoFrame and calling VideoFrame::copyFrom causes an invalid read.

Change-Id: Icb1a0a20be04d212357f5eb53d09ed08ab58cde3

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

1.8.2 causes a spurious crash, 1.8.4 doesn't.

Change-Id: I57b91759941c3f4f858d68ea59503b3320f5a8ae

* Use searchForIgd_ as an atomic flag for starting IGD search.
* Avoid locking unnecessarly in pupnp callback

Change-Id: I9a6e626945307bc9893d45ba8db0c5c9690458a8

Pierre Lespagnol authored 5 years ago and Philippe Gorley committed 5 years ago

Limit key frame request to one per second

Change-Id: Id32c710ac44c017bc222dffb21f79b6e61af2534

Pierre Lespagnol authored 5 years ago and Philippe Gorley committed 5 years ago

Change-Id: I37aedae1cd6f128ac5bf93650bc462f0d8be0e95

Philippe Gorley authored 5 years ago and Philippe Gorley committed 5 years ago

FFmpeg's AVOption API is more flexible and better suited than
AVDictionary for codec and format options.

This also allows to use strings in encoder.json, such as setting profile
to high (instead of 100).

Change-Id: I2039b2d039a87f8b3ed8ecb197dfa7b734c27950

Change-Id: Ic7745c129827bc33062d5b428fe9e376a1b22b59

Setting probesize to 1 has negative impact on file streaming.
Libavformat exposes fpsprobesize option, so we can still keep a minimal
delay before receiving video.

Change-Id: I4121dbbf46b11ed843d78cc148306fba84c1cedc

Because several devices can answer for a transfer, we must use
separate ice transport for each transfer to avoid to re-init
the transport during another transport.

Change-Id: I1ba5561bbc331e74320d9dcd6d15cca965588fec
Gitlab: #128

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

This reverts commit 0b9dd0e6.

Change-Id: I3d42ca9673de395f8f6a36c7708e0654b638ec73

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

Change-Id: I5a7764f810d39ea3f691a821a9669d549700517c

Change-Id: I7d266d9d82d76715f8f88807e4944821a04aee20

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

This is a bit a dirty fix, but the ICE transport can use several
sockets or even an IPv4 socket on a side and an IPv6 on the other
side. But, if it's the case, the header sent contains an IPv6
address and sip_transport will return PJ_ENOTSUITABLE because
TLS6 != TLS4.

In this patch, we just ignore the V6 extension.

Change-Id: I3ded796dd6ee95a08ec54656dc2ad8107f0e62e3
Gitlab: #134

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

This mainly solve the "TLS decryption failed" error happening when
auto-answer is enabled. Because we only used only one buffer for
rx datas, if two sockets answered at the same time, the buffer was
mixing both answer into one packet.

Now, each active sock has a independant rx_buffer to separate
incoming packets.

Change-Id: I95320a10804b00a0f2558a1a26a1f219b377ddbf
Gitlab: #138

Eden Abitbol authored 5 years ago and Philippe Gorley committed 5 years ago

Add media-sdk contrib folder. Includes fetch_and_patch.bat and
SHA512SUMS files.

Modify ffmpeg contrib build to link with libmfx and enable qsv
encoders/decoders.

Change-Id: I50d4e479d4619922f98377d3588424331d12aec2

Change-Id: I23bc4b136f6c546d58ba5771d3a9336d90fff5a8

Avoid needless wakeups by waiting for 10 s instead of 100 ms.
Thread is always woken up when necessary anyway.

Change-Id: I75078a56c13a649aa81c224a2fa20defd4297b78

When setting the state directly, state change callbacks are not called.

State is now always set by the TLS handler thread, which will call state
callbacks.

Change-Id: Iffd03a2b20a3bf0527aa404914728ed1d889288b

Change-Id: I5d6c119143c6eec64724226b3a2d5ebe279e0406

Change-Id: Ic1381e5f00b386f9af801e138c86d0cee1af6abb

Ming Rui Zhang authored 5 years ago and Andreas Traczyk committed 5 years ago

- now, audio only call can be allowed to enable the previewing in the setting
- in LRC, call start startpreview

Change-Id: I671841ee663fdf50790d772ce6298d1d5bd88900

Change-Id: I8a219f22bf6dc119f2c89c38b75b0093f0622d7b

Change-Id: I4e05d780481c340e288fcbb56898f30fb40bf90c

Pierre Lespagnol authored 5 years ago and Philippe Gorley committed 5 years ago

1 rtcp packet every 5 rtp packets instead of a percentage of byte sent

Change-Id: I6baf4672840ae80031b03e3bc995221aa92f0191

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

When using nvenc and the media sender gets restarted, the peer's srtp
decryption breaks and only gets set up correctly again after a pause and
play.

This issue was traced back to the RTP sequence numbers. Skipping a few
numbers before reinitializing the sender works.

Change-Id: Ic93f1368fc7eb01b3c57341f31d3832c9f2f2c1f

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

Also removes log message.

Change-Id: I5fc4dff440641f604f7f522d8319523715f4ef89

Change-Id: Ib925c5d60c1d4700e8ba2e06d2469daff2350072

Pierre Lespagnol authored 5 years ago and Philippe Gorley committed 5 years ago

reduce drop threshold and diminution rate

Change-Id: I651e3f730b638c1e94fca3882ef69f2f9e0e7732

220 kbit/s to 600 kbit/s

Change-Id: Ib6a1fb9c84f5da7a8fd2b83d113621a131d48f00

Philippe Gorley authored 5 years ago and Adrien Béraud committed 5 years ago

Don't choke the network connection with multiple keyframes in
succession. Only 1 keyframe should be necessary.

Change-Id: I6c1bcc4c3973fd6cb7eb58a9928e823bc0c5fcf7

avformat_find_stream_info takes a few seconds to compute the
real/average framerate if it is not in the metadata. We don't
need this, so we can safely skip it.

Change-Id: I134b6f478e4275c3588788c475cdfb630f3f76b5