From 0f4c14a92e590cec6ffb097430a614f0af264bc0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Blin?=
<sebastien.blin@savoirfairelinux.com>
Date: Thu, 29 Apr 2021 17:45:58 -0400
Subject: [PATCH] jamiaccount: refresh ips once connected and fix log
Change-Id: I50e2e728a679e44dcfc1903b5ac011d689910bf1
---
src/jamidht/jamiaccount.cpp | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/jamidht/jamiaccount.cpp b/src/jamidht/jamiaccount.cpp
index 2223e8e63f..70b94c2abf 100644
--- a/src/jamidht/jamiaccount.cpp
+++ b/src/jamidht/jamiaccount.cpp
@@ -2308,15 +2308,12 @@ JamiAccount::doRegister_()
auto newStatus = std::max(s4, s6);
switch (newStatus) {
case dht::NodeStatus::Connecting:
- JAMI_WARN("[Account %s] connecting to the DHT network...", getAccountID().c_str());
state = RegistrationState::TRYING;
break;
case dht::NodeStatus::Connected:
- JAMI_WARN("[Account %s] connected to the DHT network", getAccountID().c_str());
state = RegistrationState::REGISTERED;
break;
case dht::NodeStatus::Disconnected:
- JAMI_WARN("[Account %s] disconnected from the DHT network", getAccountID().c_str());
state = RegistrationState::UNREGISTERED;
break;
default:
@@ -2872,9 +2869,15 @@ JamiAccount::setRegistrationState(RegistrationState state,
const std::string& detail_str)
{
if (registrationState_ != state) {
- // Store ip whatever status changes.
- cacheTurnServers();
- storeActiveIpAddress();
+ if (state == RegistrationState::REGISTERED) {
+ JAMI_WARN("[Account %s] connected", getAccountID().c_str());
+ cacheTurnServers();
+ storeActiveIpAddress();
+ } else if (state == RegistrationState::TRYING) {
+ JAMI_WARN("[Account %s] connecting…", getAccountID().c_str());
+ } else {
+ JAMI_WARN("[Account %s] disconnected", getAccountID().c_str());
+ }
}
// Update registrationState_ & emit signals
Account::setRegistrationState(state, detail_code, detail_str);
--
GitLab