From afd6deea86b052e933f2e7f6c13cb79f3bd59cd8 Mon Sep 17 00:00:00 2001 From: Valere Date: Mon, 15 Apr 2024 10:00:19 +0200 Subject: [PATCH] review use array last for safety --- Riot/Modules/Analytics/DecryptionFailureTracker.swift | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Riot/Modules/Analytics/DecryptionFailureTracker.swift b/Riot/Modules/Analytics/DecryptionFailureTracker.swift index 0832476ffe..d76a38ca59 100644 --- a/Riot/Modules/Analytics/DecryptionFailureTracker.swift +++ b/Riot/Modules/Analytics/DecryptionFailureTracker.swift @@ -103,12 +103,12 @@ class DecryptionFailureTracker: NSObject { failure.eventLocalAgeMillis = Int(exactly: eventRelativeAgeMillis) failure.trustOwnIdentityAtTimeOfFailure = isSessionVerified - let myDomain = userId.components(separatedBy: ":")[1] + let myDomain = userId.components(separatedBy: ":").last failure.isMatrixOrg = myDomain == "matrix.org" if MXTools.isMatrixUserIdentifier(event.sender) { - let senderDomain = event.sender.components(separatedBy: ":")[1] - failure.isFederated = senderDomain != myDomain + let senderDomain = event.sender.components(separatedBy: ":").last + failure.isFederated = senderDomain != nil && senderDomain != myDomain } /// XXX for future work, as for now only the event formatter reports UTDs. That means that it's only UTD ~visible to users