Follow up to https://review.jami.net/c/dhtnet/+/30806

Change-Id: I0b7367cb5329976a8df474315d18b6f25194585d

Removed the packaging documentation from the main README and added a packaging section next to the tools sections.

The main README should be only about the dhtnet library, while the packaging documentation is part of the tools documentation, as it mainly focuses on the dnc server and client.

Change-Id: Ida8a1dab34dedb095d41b6444d882d0a34e14d52

Avoids blocking on socket write sequentially,
and prevents locking while blocked on a write.

Change-Id: Ie9444916cd5414a4379178560b316327c8a1bb67

Adrien Béraud authored 1 month ago and Adrien Béraud committed 1 month ago

Change-Id: I6bb77810a26c763e90ae889781cb1a3721b9cce3

Amna Snene authored 8 months ago and Adrien Béraud committed 1 month ago

Change-Id: Iac9785c81df32c4373faab71e931d8f8fa3ffbc6

Change-Id: I6053e24e501513fa444af08367d423b4323763c0

Adrien Béraud authored 1 month ago and Adrien Béraud committed 1 month ago

Change-Id: I7e35f5d33beba65134ef4d01199b8bc8f3697aee

Adrien Béraud authored 1 month ago and Adrien Béraud committed 1 month ago

Change-Id: I39a23e0e983716bb5fd6ac615aca57cd7d2575c1

The settings (host, username, password, realm) required to use TURN with
dsh were passed to the Dsh constructor but not set in the
ConnectionManger config.

GitLab: #634
Change-Id: Ib6619265fda1d20be83032f83798ea92d13616fc

Change-Id: I392d74df1c97654a16ff6ce9b30ab6557ebe0c7d

Change-Id: I224d498d4b1bc41eccb4a0e81afafda80dd08cda

ovari authored 2 months ago and Adrien Béraud committed 2 months ago

Change-Id: I42b10e4025d2b37c2c377e874bc5abb1005b6e6c

ovari authored 2 months ago and ovari committed 2 months ago

Change-Id: I8a9c0e958b39296968837de502e81a0a0d3f623d

Change-Id: I10597a207dcca24ead8e936680c60af7eb36bc03

Change-Id: I0c093ea4e1834866fd08abb30d51d5043fe1f455

This patch removes the custom retry limit and timeout in the
NatPmp::readResponse function; they were not necessary given that
libnatpmp already implements this functionality.

This change means that readResponse can no longer return the
NATPMP_TRYAGAIN error code. The fact that it could previously seems to
be what enabled the bug described in the following issue:
jami-client-qt#1898
More precisely, it seems that the readResponse function was repeatedly
returning NATPMP_TRYAGAIN (even though the mapping requests had actually
succeeded), which isn't a "fatal" error (as determined by the
NatPmp::isErrorFatal function) and therefore didn't cause the error
counter to be incremented and the IGD to be invalidated.

GitLab: #43
Change-Id: I1374ba1b40e867399dab0f2dc4f6f75424f4a6b6

This patch fixes a bug where FAILED mappings with the "auto-update"
option enabled are left in the local list, even after a new mapping has
been requested. This can cause the list to grow substantially over time
and was partly responsible for the bug described in the following issue:
jami-client-qt#1898

GitLab: #45
Change-Id: I009ff791ae90d10f64d46dede3b2cbfc9c1b7160

GitLab: #632
Change-Id: I54a9e41f1cd026183a0fa3aa0891e49e99785517

authentification → authentication
chunck → chunk
DTLS
GnuTLS
IPv4
IPv6
MiM attack → MITM attack
MTU
non-reliable → unreliable
non-valid → invalid
OCSP
OK
Renogotiate → Renegotiate
retry → reattempt
shuting → shutting
tried → attempt
try → attempt
URI
x509 → X.509

Change-Id: I5ab67f1fb10b24aa8de79217cba219137de6a88b

Change-Id: Id59bce284de0a8a683c62d5eb4388fe228f4d950

I/O queue
ID
intermidiate → intermediate
IP address
IPv4
IPv6
not initialized → Uninitialized
STUN
too much → Too many

Change-Id: Ice56bef0cf363ce676f516ed32f475c9cd248b15

…

Change-Id: I8dfaa1fab2dd712bc09d9689b9eaf657b72a2c5e

Gitlab: #631
Change-Id: I17b5e5edd7bf4d7a7ce26d766146d796d41c8889

The UPnPContext class currently has hardcoded limits on the number of
mappings that need to be "available" at any given time. This patch adds
the ability to modify these limits and sets them to zero in the upnpctrl
tool. This changes the behavior of the tool in two ways:
1) setting the minimum limit to zero means that no mappings are opened
   when upnpctrl starts;
2) setting the maximum limit to zero means that the "close" command will
   always cause the relevant mapping to be closed immediately instead of
   (potentially) being put back in the list of available mappings.

Change-Id: Idcf4262ce80a8b39f19ce7d3a201b40b3ea56911

... → …
wish → want

Change-Id: Iedfd8fcf6cdb56819b80a954d511f83eb2b8e906

ssh → SSH
stun server → STUN server
try → attempt
turn server → TURN server
UPNP → UPnP

Change-Id: Ia31a3baae908850a7e99d50128447f2eb9113df1

ovari authored 4 months ago and Adrien Béraud committed 4 months ago

won't → unable to

Change-Id: I1ce5af3ac16ed364df62d20338507b714a47b913
GitLab: jami-client-qt#1730

platform → platforms

Change-Id: I99a79c2073e4c9a593fd264f75e22e4e1df1afcc
GitLab: jami-client-qt#1730

{can't, won't} → unable to
plateform → platform

Change-Id: I7723ac5ed881572201fb8367f1e5c84c52330888
GitLab: jami-client-qt#1730

Change-Id: I3acce7150d5f21bfdaf6174060e8671086c4f56e

3-character ellipsis ... → 1-character ellipsis …

Change-Id: If14c822aaf7c32a50a5367c26cc2deededebfd90
GitLab: jami-client-qt#1730

Adrien Béraud authored 4 months ago and Adrien Béraud committed 4 months ago

Change-Id: Ied3cc750a1dca7dd14f3ef75ffc8729c09bf7087

* fix: use the same mutex in callback and to wait
* fix: notify with mutex locked
* fix: avoid accessing this from callback, preventing potential use-after-free
* allows to stop waiting for upnp mappings immediately
 when the transport is stopped
* perform single allocation of shared state,
 and free the memory of mutex and cv after mapping completes

Change-Id: I573937d2e8a90b31501a237607bbc7f52942bcea

Change-Id: Ic7c5bf97c9ac159519eea2c265787cf47d378ce7

On Ubuntu/debian, it have happenned that some build have inconsistent
paths like /usr/usr/lib. To avoid this, hard-coded most of the paths
like we are doing in Fedora.

Change-Id: I8c1a3567490e02ac11ec57315bc3a7cc38d02ea0

Cleaned all the dead code in debian/ubuntu post install script.

Change-Id: Idb23b5974e544d704ee9feb753990bf185575ec5

Building dhtnet for ubuntu 24.10 require the addition of libzstd for
compilation to work (linker require it).

Change-Id: I2168a84ee9fde3dc21925f000d5f136ef76fb207

Louis Maillard authored 5 months ago and Adrien Béraud committed 5 months ago

Add dhtnet.spec for RPM build and fedora_40,39 dockerfile for env build.
Fix and changes was required in Cmake instruction in order to build:
- Dependencies can now be built in static mode even with dynamic library
- Remove occurence of INSTALL_DIR path in dhtnet.pc and dnc.service
  but only when option TRIM_PREFIX_PATH is turned On.

Fix and replace previously-reverted review 28703
Replace change id (I2f2161d7c65241a35c4acbcc006ba27bfcc1f176)

Change-Id: I5a80317d83791743ba6ee443fae7f89156e8013b

Change-Id: Ifef17404e31f7478d310179255092d0e17743b16

Only process the callback if it was not already done by the tls
shutdown callback.

Change-Id: Iee9b479100268c4815f40f7b5d51b9db7dbb22cd