From 4eb504d000454d750da7ac4a2589394abe3d1e90 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Damir=20Jeli=C4=87?= <poljar@termina.org.uk>
Date: Mon, 18 Jan 2021 14:15:31 +0100
Subject: [PATCH] crypto: Improve the log line when we share group sessions

---
 .../src/session_manager/group_sessions.rs     | 22 +++++++++++--------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/matrix_sdk_crypto/src/session_manager/group_sessions.rs b/matrix_sdk_crypto/src/session_manager/group_sessions.rs
index e63985ff..9f4da393 100644
--- a/matrix_sdk_crypto/src/session_manager/group_sessions.rs
+++ b/matrix_sdk_crypto/src/session_manager/group_sessions.rs
@@ -21,7 +21,7 @@ use dashmap::DashMap;
 use matrix_sdk_common::{
     api::r0::to_device::DeviceIdOrAllDevices,
     events::{room::encrypted::EncryptedEventContent, AnyMessageEventContent, EventType},
-    identifiers::{DeviceId, DeviceIdBox, RoomId, UserId},
+    identifiers::{DeviceIdBox, RoomId, UserId},
     uuid::Uuid,
 };
 use tracing::{debug, info};
@@ -236,14 +236,18 @@ impl GroupSessionManager {
             .flatten()
             .collect();
 
-        info!(
-            "Sharing outbound session at index {} with {:?}",
-            outbound.message_index().await,
-            devices
-                .iter()
-                .map(|d| (d.user_id(), d.device_id()))
-                .collect::<Vec<(&UserId, &DeviceId)>>()
-        );
+        if !devices.is_empty() {
+            info!(
+                "Sharing outbound session at index {} with {:?}",
+                outbound.message_index().await,
+                devices.iter().fold(HashMap::new(), |mut acc, d| {
+                    acc.entry(d.user_id())
+                        .or_insert_with(Vec::new)
+                        .push(d.device_id());
+                    acc
+                })
+            );
+        }
 
         let key_content = outbound.as_json().await;