In a one to one swarm, the invited file is not always the account.
Fix regression from 588890f9

Change-Id: I11581f37e2b7829bf05a42cfd48ab146bbb735f3

Change-Id: Ia052e6a86c3cb68c9895b0ee87e96391990b1d72

Change-Id: Ic8264b3b2b6f9cc3c4f92274830c49ed7fe3f87c

Change-Id: I1eba92ae040feafacad7c282c785f890704fc6b0

parseAnnounce doesn't use the longId. This fix authentification
to JAMS accounts.

Change-Id: I87a6b3d5893bb3aae6c814b086f50cd023d8a85c
GitLab: #579

Olivier Dion authored 3 years ago and Sébastien Blin committed 3 years ago

Change-Id: Ic1f59314e62690cd7976095a8ec2e0cad49ccfc6

This fix ut_compability and a regression from
95280b19

Change-Id: I4a41e3c383fd8c8d31e844d2cd58fa82ef04fd00
GitLab: #579

Change-Id: Ie9f478783af7275eb360e3098d0e0d4bae7018e8

A user can remove a contact while a conversation is syncing with
this contact. In this case, the conversation was not deleted
properly. Causing some UI bug when re-adding the contact.

GitLab: #596
Change-Id: I88f2549a02ba67e9030cdadfdfb45f34a0ee06b9

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

Change-Id: I5e135b9ef4365acf87e6de4a8b1af9d19ef34ca3

Olivier Dion authored 3 years ago and Adrien Béraud committed 3 years ago

By removing the YAML configuration, the agent is now configured via a Scheme
file interpreted by Guile.

There's no need for behavior trees now since it can be reproduced naturally in
the Scheme language.

See README.md for how to compile and run.

Change-Id: Ibbf9ed0f965cee71bce63c0f8e8c12862f37176d

This breaks one to one swarm. Now, invited/ is added to detect
contacts that joins and left a conversation. This is wanted
to show read-only conversations.

Change-Id: I519094e75872062538fe56dd3a32378da814005b
GitLab: #595

With this patch, the daemon is now storing informations based
on setMessageDisplayed/onMessageDisplayed to follow the read
status per participant.

This avoid to creates a logic in all clients and allow the client
to have a correct read status even if the daemon was launched
separately.
Clients can now:

+ Retrieve last displayed interactions by retrieving members infos
(via getConversationMembers() and the key "lastDisplayed")
+ Know how many messages there between the head and a commit. This
is useful to count unread messages for a peer.

GitLab: #592
Change-Id: I0b5bc7adb5cda48754f33a362a40ce10ee345e3c

This changes tends to simplify the client code to
detect syncing conversations. Note, this patch
introduces some breaking changes for the stored
convInfos, but this will be fixed when all devices
are up-to-date.

Changes:

+ getConversations() now returns conversations as soon
as the request is accepted.
+ getConversationMembers() return members for syncing
conversation (current account and the sender of the
request)
+ "convInfo" now stores a map<string, ConvInfo> to
avoid iterating over all conversations. This changes
the structure stored on the disk and sent during the sync.

GitLab: #591
Change-Id: I365875cda89c483bcf9c6ea3bd8222e6253773b1

Change-Id: Icb4266aeb1274c02d8fc7e60dabfc7a7bec7a17e

Change-Id: I74f8b2cdb8cdf258022f32478dba8dad9e54353c

Olivier Dion authored 3 years ago and Sébastien Blin committed 3 years ago

Change-Id: I8c834ea5abefc1f61996ab5098f1d7caf74e36dc

Change-Id: I512c6a5f2fe905d0857a5f22488cf01596a983b5

Change-Id: Iac966cdac5b6c817cf6b074147830929b1c7283c

Mohamed Chibani authored 3 years ago and Sébastien Blin committed 3 years ago

When a new media list is provided (during a media change request
for instance), the media source and source type will be updated
only if set in the new media attribute list. This will prevent unwanted
overwriting of media source and source type.
This patch will fix the ut_media_negotiation unit test.

Gitlab: #576

Change-Id: If6f994def1e113265eeff0cce93fe5d314dffd5d

If the user decides to transmit its account archive via a webservice
in some case the webserver can re-compress the file resulting to a
gunzip file into a gunzip file. We can check the header of the file
to be able to detect this case.

Then, replace inflateInit by inflateInit2 with better header detection
to avoid header errors. Cf http://www.zlib.net/manual.html#inflateInit2

Change-Id: I1421affa8815ab347b439ef2e505da10275c80ff
GitLab: #575

Change-Id: I5340a0215512be416a618c526f8a0962c8401087

Gitlab: #590

Change-Id: Ia2f030e475eb269a7f9ccb7540fd1fc0c80f5a5b

Sébastien Blin authored 3 years ago and Olivier Dion committed 3 years ago

Note:

Merged upstream (so deleted):
0010-fix-pkgconfig.patch
0014-Add-new-compile-time-setting-PJ_ICE_ST_USE_TURN_PERM.patch
0015-update-local-preference-for-peer-reflexive-candidate.patch
0016-use-addrinfo-instead-CFHOST.patch
0017-CVE-2020-15260.patch
0018-CVE-2021-21375.patch

Merged for 2.12:
0019-ignore-down-interfaces.patch
0020-ignore-addresses-for-RFC7335.patch

Also, fix the assertion in ice_transport.cpp cause
user_mapping_cnt can be equals to PJ_ICE_MAX_COMP.

Change-Id: If6fee261e89c34640519a6b61e94391b57363a22
GitLab: #509

Change-Id: Idab58d884268f94048c0ca52adb2bb5397aac95f

Change-Id: I72147bfa7fcf06d69a9cd911300ce3dada1f2649

Change-Id: I78e539ba9702bba09afb0f6c3bacd115e3fde737

Fixing a deadlock that can occur with this path:
startStream -> waitForDevices (blocks with mutex locked) -> hardwareFormatAvailable (deadlock on mutex)

Change-Id: I808513e62618986b9a4217d3e70f57cb91cef595

Change-Id: I48a46f4dacd78f259be79547a6d8e4a02ae1c5f5

If `DRing::acceptWithMedia()` is called within an `IncomingCall` signal handler,
then `processIncomingCall` will wrongly try to answer the same call.

Gitlab: #589

Change-Id: Ica7a5c17852d83e6228e71e0f9a2a83b24cb315b

When a call is added to a conference, the control of the mute/un-mute
state of the media of the call is taken over by the conference, and
the mute state of the participating calls will be controlled by the
state of the local host set in the conference, which basically consists
of attaching/detaching the source to/from the mixer. Currently the local
host mute state might not be correctly initialized, leading to
inconsistent mute states.
The proposed changes will correctly set the local host state according
to the initial mute states of each call joining the conference.

Gitlab: #576

Change-Id: I0a746aae82da57222cc7ff91c2e39a1a2bbaff8e

Gitlab: #567

Change-Id: I7f9eb1f9eb13250a999f7b26441ea7c47b81f796

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

verifyOcsp takes a callback with arguments passed by reference.
However, if the method finishes (verifyCertificateWrapper) cert
will be destroyed and references potentially invalidated.
So, instead of waiting on a condition variable use a future to
be sure that the cert is used before destruction.
The callback is always called by sendOcspRequest with a similar
timeout so also remove the useless timeout.

Change-Id: If1a8eba70e13d2d613b758ad24629efa4fe57bb3
GitLab: #577

Dolt doesn't provide the performance benefits it promises, so drop it
to simplify the moving parts of the Autotools build system.

Also remove unused M4 modules.

* m4/dolt.m4: Delete file.
* m4/ax_boost_base.m4: Likewise.
* configure.ac: Do not call DOLT.

Change-Id: I58d286457005ad82129e089679c52933989c59a8

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

Change-Id: Ia395103ba2ec2ec85400cf710a07c40888568d68

For whatever reason, a conversation can be malformed. This should
be detected and the conversation not loaded. The conversation
will be removed and re-synced if necessary.

Change-Id: I40e78564559e7f09159ceba4d263a4ea75b25794

Maxim Cournoyer authored 3 years ago and Maxim Cournoyer committed 3 years ago

Automated using the following commands:

  $ mv src/{dring,jami}
  $ git grep -l src/dring | xargs sed -i 's,src/dring,src/jami,g'
  $ git grep -l '#include "dring/' | \
    xargs sed -i 's,#include "dring/,#include "jami/,g'
  $ git grep -l 'dring.h' | xargs sed -i 's,dring.h,jami.h,g'

And finally,

  $ git grep -l 'dring' | xargs sed -i 's,dring,jami,g'

  $ files=$(find -name '*dring*' | sort)
  $ for f in $files; do mkdir -p "$(dirname "$f")"; \
      mv "$f" "$(echo $f | sed 's/dring/jami/g')"; done

To resolve a bad renaming favorably:

  $ git grep -l -i AlsaCarjami | \
    xargs sed -i -E 's/([Aa])lsaCarjami/\1lsaCardRingtone/g'

The above renaming command is not perfect, so some hand-tuning was
required to complete it.

* src/manager.cpp (Manager::ManagerPimpl::retrieveConfigPath):
Preserve the dring.yml configuration file name, until we add something
to migrate (rename) it to jami.yml.
* man/dring.pod: Delete.
* bin/dbus/jamid.pod: Move to ...
* man/jamid.pod: here.
* bin/dbus/meson.build (jamid_targets): Normalize man section to the
pre-existing 1 and adjust accordingly.
* src/jami/def.h (dring_EXPORTS): Rename to ...
(jami_EXPORTS): ... this.

change-Id: I9828be6da9c711ab2f22c4d1b9539fea89d7b6fb

This logic was added via 9d9a6b3d,
but this will give potential deadlocks, because pj's group lock
will be locked at this point. So we will have:
group-lock->callMutex_
and other methods like SIPCall::hangup will try to lock:
callMutex_->group-lock
causing a deadlock.

The original issue was adding this line due to a use after free,
however since then we fixed the ref counter of inv, so it should
be a valid fix.

Change-Id: I2fdc236ca63487634ebf2659ecdebb8e0c3234fe
GitLab: #586

Change-Id: I33c732784833808d092cefa402669a0bbdc0561e
GitLab: #582

Gitlab: #583

Change-Id: Iddd1b582595daee498b383a11c6323546d14bef4