diff --git a/bitchat/Identity/SecureIdentityStateManager.swift b/bitchat/Identity/SecureIdentityStateManager.swift index fecdc793..4c98dfaf 100644 --- a/bitchat/Identity/SecureIdentityStateManager.swift +++ b/bitchat/Identity/SecureIdentityStateManager.swift @@ -160,7 +160,7 @@ final class SecureIdentityStateManager { cache = try JSONDecoder().decode(IdentityCache.self, from: decryptedData) } catch { // Log error but continue with empty cache - SecureLogger.logError(error, context: "Failed to load identity cache", category: SecureLogger.security) + SecureLogger.logError(error, context: "Failed to load identity cache", category: .security) } } @@ -191,10 +191,10 @@ final class SecureIdentityStateManager { let sealedBox = try AES.GCM.seal(data, using: encryptionKey) let saved = keychain.saveIdentityKey(sealedBox.combined!, forKey: cacheKey) if saved { - SecureLogger.log("Identity cache saved to keychain", category: SecureLogger.security, level: .debug) + SecureLogger.log("Identity cache saved to keychain", category: .security, level: .debug) } } catch { - SecureLogger.logError(error, context: "Failed to save identity cache", category: SecureLogger.security) + SecureLogger.logError(error, context: "Failed to save identity cache", category: .security) } } @@ -395,7 +395,7 @@ final class SecureIdentityStateManager { } func setBlocked(_ fingerprint: String, isBlocked: Bool) { - SecureLogger.log("User \(isBlocked ? "blocked" : "unblocked"): \(fingerprint)", category: SecureLogger.security, level: .info) + SecureLogger.log("User \(isBlocked ? "blocked" : "unblocked"): \(fingerprint)", category: .security, level: .info) queue.async(flags: .barrier) { if var identity = self.cache.socialIdentities[fingerprint] { @@ -519,7 +519,7 @@ final class SecureIdentityStateManager { // MARK: - Cleanup func clearAllIdentityData() { - SecureLogger.log("Clearing all identity data", category: SecureLogger.security, level: .warning) + SecureLogger.log("Clearing all identity data", category: .security, level: .warning) queue.async(flags: .barrier) { self.cache = IdentityCache() @@ -543,7 +543,7 @@ final class SecureIdentityStateManager { // MARK: - Verification func setVerified(fingerprint: String, verified: Bool) { - SecureLogger.log("Fingerprint \(verified ? "verified" : "unverified"): \(fingerprint)", category: SecureLogger.security, level: .info) + SecureLogger.log("Fingerprint \(verified ? "verified" : "unverified"): \(fingerprint)", category: .security, level: .info) queue.async(flags: .barrier) { if verified { diff --git a/bitchat/Noise/NoiseHandshakeCoordinator.swift b/bitchat/Noise/NoiseHandshakeCoordinator.swift index 0a14c273..c22406d6 100644 --- a/bitchat/Noise/NoiseHandshakeCoordinator.swift +++ b/bitchat/Noise/NoiseHandshakeCoordinator.swift @@ -69,7 +69,7 @@ final class NoiseHandshakeCoordinator { case .initiating(_, let lastAttempt): if Date().timeIntervalSince(lastAttempt) > handshakeTimeout { SecureLogger.log("Forcing new handshake with \(remotePeerID) - previous stuck in initiating", - category: SecureLogger.handshake, level: .warning) + category: .handshake, level: .warning) return true } default: @@ -78,7 +78,7 @@ final class NoiseHandshakeCoordinator { } SecureLogger.log("Already in active handshake with \(remotePeerID), state: \(state)", - category: SecureLogger.handshake, level: .debug) + category: .handshake, level: .debug) return false } @@ -108,7 +108,7 @@ final class NoiseHandshakeCoordinator { let attempt = self.getCurrentAttempt(for: peerID) + 1 self.handshakeStates[peerID] = .initiating(attempt: attempt, lastAttempt: Date()) SecureLogger.log("Recording handshake initiation with \(peerID), attempt \(attempt)", - category: SecureLogger.handshake, level: .info) + category: .handshake, level: .info) } } @@ -117,7 +117,7 @@ final class NoiseHandshakeCoordinator { handshakeQueue.async(flags: .barrier) { self.handshakeStates[peerID] = .responding(since: Date()) SecureLogger.log("Recording handshake response to \(peerID)", - category: SecureLogger.handshake, level: .info) + category: .handshake, level: .info) } } @@ -126,7 +126,7 @@ final class NoiseHandshakeCoordinator { handshakeQueue.async(flags: .barrier) { self.handshakeStates[peerID] = .established(since: Date()) SecureLogger.log("Handshake successfully established with \(peerID)", - category: SecureLogger.handshake, level: .info) + category: .handshake, level: .info) } } @@ -137,7 +137,7 @@ final class NoiseHandshakeCoordinator { let canRetry = attempts < self.maxHandshakeAttempts self.handshakeStates[peerID] = .failed(reason: reason, canRetry: canRetry, lastAttempt: Date()) SecureLogger.log("Handshake failed with \(peerID): \(reason), canRetry: \(canRetry)", - category: SecureLogger.handshake, level: .warning) + category: .handshake, level: .warning) } } @@ -147,7 +147,7 @@ final class NoiseHandshakeCoordinator { // If we're already established, reject new handshakes if case .established = handshakeStates[remotePeerID] { SecureLogger.log("Rejecting handshake from \(remotePeerID) - already established", - category: SecureLogger.handshake, level: .debug) + category: .handshake, level: .debug) return false } @@ -158,7 +158,7 @@ final class NoiseHandshakeCoordinator { if case .initiating = handshakeStates[remotePeerID] { // They shouldn't be initiating, but accept it to recover from race condition SecureLogger.log("Accepting handshake from \(remotePeerID) despite being initiator (race condition recovery)", - category: SecureLogger.handshake, level: .warning) + category: .handshake, level: .warning) return true } } @@ -216,7 +216,7 @@ final class NoiseHandshakeCoordinator { handshakeQueue.async(flags: .barrier) { self.handshakeStates.removeValue(forKey: peerID) SecureLogger.log("Reset handshake state for \(peerID)", - category: SecureLogger.handshake, level: .debug) + category: .handshake, level: .debug) } } @@ -257,7 +257,7 @@ final class NoiseHandshakeCoordinator { if isStale { stalePeerIDs.append(peerID) SecureLogger.log("Found stale handshake state for \(peerID): \(state)", - category: SecureLogger.handshake, level: .warning) + category: .handshake, level: .warning) } } @@ -271,7 +271,7 @@ final class NoiseHandshakeCoordinator { let peerID = sortedSessions[i].peerID stalePeerIDs.append(peerID) SecureLogger.log("Removing old established session for \(peerID) to maintain session limit", - category: SecureLogger.handshake, level: .info) + category: .handshake, level: .info) } } @@ -282,7 +282,7 @@ final class NoiseHandshakeCoordinator { if !stalePeerIDs.isEmpty { SecureLogger.log("Cleaned up \(stalePeerIDs.count) stale handshake states", - category: SecureLogger.handshake, level: .info) + category: .handshake, level: .info) } return stalePeerIDs @@ -333,7 +333,7 @@ final class NoiseHandshakeCoordinator { /// Log current handshake states for debugging func logHandshakeStates() { handshakeQueue.sync { - SecureLogger.log("=== Handshake States ===", category: SecureLogger.handshake, level: .debug) + SecureLogger.log("=== Handshake States ===", category: .handshake, level: .debug) for (peerID, state) in handshakeStates { let stateDesc: String switch state { @@ -352,18 +352,18 @@ final class NoiseHandshakeCoordinator { case .failed(let reason, let canRetry, let lastAttempt): stateDesc = "failed: \(reason) (canRetry: \(canRetry), last: \(lastAttempt))" } - SecureLogger.log(" \(peerID): \(stateDesc)", category: SecureLogger.handshake, level: .debug) + SecureLogger.log(" \(peerID): \(stateDesc)", category: .handshake, level: .debug) } - SecureLogger.log("========================", category: SecureLogger.handshake, level: .debug) + SecureLogger.log("========================", category: .handshake, level: .debug) } } /// Clear all handshake states - used during panic mode func clearAllHandshakeStates() { handshakeQueue.async(flags: .barrier) { - SecureLogger.log("Clearing all handshake states for panic mode", category: SecureLogger.handshake, level: .warning) + SecureLogger.log("Clearing all handshake states for panic mode", category: .handshake, level: .warning) self.handshakeStates.removeAll() self.processedHandshakeMessages.removeAll() } } -} \ No newline at end of file +} diff --git a/bitchat/Noise/NoiseProtocol.swift b/bitchat/Noise/NoiseProtocol.swift index 5725e5e4..f3bd3d4f 100644 --- a/bitchat/Noise/NoiseProtocol.swift +++ b/bitchat/Noise/NoiseProtocol.swift @@ -285,7 +285,7 @@ final class NoiseCipherState { // Log high nonce values that might indicate issues if currentNonce > Self.HIGH_NONCE_WARNING_THRESHOLD { - SecureLogger.log("High nonce value detected: \(currentNonce) - consider rekeying", category: SecureLogger.encryption, level: .warning) + SecureLogger.log("High nonce value detected: \(currentNonce) - consider rekeying", category: .encryption, level: .warning) } return combinedPayload @@ -342,7 +342,7 @@ final class NoiseCipherState { // Log high nonce values that might indicate issues if decryptionNonce > Self.HIGH_NONCE_WARNING_THRESHOLD { - SecureLogger.log("High nonce value detected: \(decryptionNonce) - consider rekeying", category: SecureLogger.encryption, level: .warning) + SecureLogger.log("High nonce value detected: \(decryptionNonce) - consider rekeying", category: .encryption, level: .warning) } do { @@ -357,7 +357,7 @@ final class NoiseCipherState { } catch { SecureLogger.log("Decrypt failed: \(error) for nonce \(decryptionNonce)") // Log authentication failures with nonce info - SecureLogger.log("Decryption failed at nonce \(decryptionNonce)", category: SecureLogger.encryption, level: .error) + SecureLogger.log("Decryption failed at nonce \(decryptionNonce)", category: .encryption, level: .error) throw error } } @@ -661,7 +661,7 @@ final class NoiseHandshakeState { do { remoteEphemeralPublic = try NoiseHandshakeState.validatePublicKey(ephemeralData) } catch { - SecureLogger.log("Invalid ephemeral public key received", category: SecureLogger.security, level: .warning) + SecureLogger.log("Invalid ephemeral public key received", category: .security, level: .warning) throw NoiseError.invalidMessage } symmetricState.mixHash(ephemeralData) @@ -877,7 +877,7 @@ extension NoiseHandshakeState { // Check against known bad points if lowOrderPoints.contains(keyData) { - SecureLogger.log("Low-order point detected", category: SecureLogger.security, level: .warning) + SecureLogger.log("Low-order point detected", category: .security, level: .warning) throw NoiseError.invalidPublicKey } @@ -887,7 +887,7 @@ extension NoiseHandshakeState { return publicKey } catch { // If CryptoKit rejects it, it's invalid - SecureLogger.log("CryptoKit validation failed", category: SecureLogger.security, level: .warning) + SecureLogger.log("CryptoKit validation failed", category: .security, level: .warning) throw NoiseError.invalidPublicKey } } diff --git a/bitchat/Noise/NoiseSecurityConsiderations.swift b/bitchat/Noise/NoiseSecurityConsiderations.swift index e4e0afc0..927b1cbf 100644 --- a/bitchat/Noise/NoiseSecurityConsiderations.swift +++ b/bitchat/Noise/NoiseSecurityConsiderations.swift @@ -153,7 +153,7 @@ final class NoiseRateLimiter { // Check global rate limit first globalHandshakeTimestamps = globalHandshakeTimestamps.filter { $0 > oneMinuteAgo } if globalHandshakeTimestamps.count >= NoiseSecurityConstants.maxGlobalHandshakesPerMinute { - SecureLogger.log("Global handshake rate limit exceeded: \(globalHandshakeTimestamps.count)/\(NoiseSecurityConstants.maxGlobalHandshakesPerMinute) per minute", category: SecureLogger.security, level: .warning) + SecureLogger.log("Global handshake rate limit exceeded: \(globalHandshakeTimestamps.count)/\(NoiseSecurityConstants.maxGlobalHandshakesPerMinute) per minute", category: .security, level: .warning) return false } @@ -162,7 +162,7 @@ final class NoiseRateLimiter { timestamps = timestamps.filter { $0 > oneMinuteAgo } if timestamps.count >= NoiseSecurityConstants.maxHandshakesPerMinute { - SecureLogger.log("Per-peer handshake rate limit exceeded for \(peerID): \(timestamps.count)/\(NoiseSecurityConstants.maxHandshakesPerMinute) per minute", category: SecureLogger.security, level: .warning) + SecureLogger.log("Per-peer handshake rate limit exceeded for \(peerID): \(timestamps.count)/\(NoiseSecurityConstants.maxHandshakesPerMinute) per minute", category: .security, level: .warning) return false } @@ -182,7 +182,7 @@ final class NoiseRateLimiter { // Check global rate limit first globalMessageTimestamps = globalMessageTimestamps.filter { $0 > oneSecondAgo } if globalMessageTimestamps.count >= NoiseSecurityConstants.maxGlobalMessagesPerSecond { - SecureLogger.log("Global message rate limit exceeded: \(globalMessageTimestamps.count)/\(NoiseSecurityConstants.maxGlobalMessagesPerSecond) per second", category: SecureLogger.security, level: .warning) + SecureLogger.log("Global message rate limit exceeded: \(globalMessageTimestamps.count)/\(NoiseSecurityConstants.maxGlobalMessagesPerSecond) per second", category: .security, level: .warning) return false } @@ -191,7 +191,7 @@ final class NoiseRateLimiter { timestamps = timestamps.filter { $0 > oneSecondAgo } if timestamps.count >= NoiseSecurityConstants.maxMessagesPerSecond { - SecureLogger.log("Per-peer message rate limit exceeded for \(peerID): \(timestamps.count)/\(NoiseSecurityConstants.maxMessagesPerSecond) per second", category: SecureLogger.security, level: .warning) + SecureLogger.log("Per-peer message rate limit exceeded for \(peerID): \(timestamps.count)/\(NoiseSecurityConstants.maxMessagesPerSecond) per second", category: .security, level: .warning) return false } diff --git a/bitchat/Noise/NoiseSession.swift b/bitchat/Noise/NoiseSession.swift index c16b1017..36623010 100644 --- a/bitchat/Noise/NoiseSession.swift +++ b/bitchat/Noise/NoiseSession.swift @@ -92,7 +92,7 @@ class NoiseSession { func processHandshakeMessage(_ message: Data) throws -> Data? { return try sessionQueue.sync(flags: .barrier) { - SecureLogger.log("NoiseSession[\(peerID)]: Processing handshake message, current state: \(state), role: \(role)", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Processing handshake message, current state: \(state), role: \(role)", category: .noise, level: .debug) // Initialize handshake state if needed (for responders) if state == .uninitialized && role == .responder { @@ -103,7 +103,7 @@ class NoiseSession { remoteStaticKey: nil ) state = .handshaking - SecureLogger.log("NoiseSession[\(peerID)]: Initialized handshake state for responder", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Initialized handshake state for responder", category: .noise, level: .debug) } guard case .handshaking = state, let handshake = handshakeState else { @@ -112,7 +112,7 @@ class NoiseSession { // Process incoming message _ = try handshake.readMessage(message) - SecureLogger.log("NoiseSession[\(peerID)]: Read handshake message, checking if complete", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Read handshake message, checking if complete", category: .noise, level: .debug) // Check if handshake is complete if handshake.isHandshakeComplete() { @@ -130,7 +130,7 @@ class NoiseSession { state = .established handshakeState = nil // Clear handshake state - SecureLogger.log("NoiseSession[\(peerID)]: Handshake complete (no response needed), transitioning to established", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Handshake complete (no response needed), transitioning to established", category: .noise, level: .debug) SecureLogger.logSecurityEvent(.handshakeCompleted(peerID: peerID)) return nil @@ -138,7 +138,7 @@ class NoiseSession { // Generate response let response = try handshake.writeMessage() sentHandshakeMessages.append(response) - SecureLogger.log("NoiseSession[\(peerID)]: Generated handshake response of size \(response.count)", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Generated handshake response of size \(response.count)", category: .noise, level: .debug) // Check if handshake is complete after writing if handshake.isHandshakeComplete() { @@ -156,7 +156,7 @@ class NoiseSession { state = .established handshakeState = nil // Clear handshake state - SecureLogger.log("NoiseSession[\(peerID)]: Handshake complete after writing response, transitioning to established", category: SecureLogger.noise, level: .debug) + SecureLogger.log("NoiseSession[\(peerID)]: Handshake complete after writing response, transitioning to established", category: .noise, level: .debug) SecureLogger.logSecurityEvent(.handshakeCompleted(peerID: peerID)) } @@ -349,7 +349,7 @@ final class NoiseSessionManager { // We should accept the new handshake to re-establish encryption if existing.isEstablished() { SecureLogger.log("Accepting handshake from \(peerID) despite existing session - peer likely cleared their session", - category: SecureLogger.session, level: .info) + category: .session, level: .info) _ = sessions.removeValue(forKey: peerID) shouldCreateNew = true } else { diff --git a/bitchat/Nostr/GeoRelayDirectory.swift b/bitchat/Nostr/GeoRelayDirectory.swift index eaf1efe9..a3269647 100644 --- a/bitchat/Nostr/GeoRelayDirectory.swift +++ b/bitchat/Nostr/GeoRelayDirectory.swift @@ -54,7 +54,7 @@ final class GeoRelayDirectory { Task.detached { let ready = await TorManager.shared.awaitReady() if !ready { - SecureLogger.log("GeoRelayDirectory: Tor not ready; skipping remote fetch (fail-closed)", category: SecureLogger.session, level: .warning) + SecureLogger.log("GeoRelayDirectory: Tor not ready; skipping remote fetch (fail-closed)", category: .session, level: .warning) return } let task = TorURLSession.shared.session.dataTask(with: req) { [weak self] data, _, error in @@ -66,12 +66,12 @@ final class GeoRelayDirectory { self.entries = parsed self.persistCache(text) UserDefaults.standard.set(Date(), forKey: self.lastFetchKey) - SecureLogger.log("GeoRelayDirectory: refreshed \(parsed.count) relays from remote", category: SecureLogger.session, level: .info) + SecureLogger.log("GeoRelayDirectory: refreshed \(parsed.count) relays from remote", category: .session, level: .info) } return } } - SecureLogger.log("GeoRelayDirectory: remote fetch failed; keeping local entries", category: SecureLogger.session, level: .warning) + SecureLogger.log("GeoRelayDirectory: remote fetch failed; keeping local entries", category: .session, level: .warning) } task.resume() } @@ -82,7 +82,7 @@ final class GeoRelayDirectory { do { try text.data(using: .utf8)?.write(to: url, options: .atomic) } catch { - SecureLogger.log("GeoRelayDirectory: failed to write cache: \(error)", category: SecureLogger.session, level: .warning) + SecureLogger.log("GeoRelayDirectory: failed to write cache: \(error)", category: .session, level: .warning) } } @@ -113,7 +113,7 @@ final class GeoRelayDirectory { let text = String(data: data, encoding: .utf8) { return Self.parseCSV(text) } - SecureLogger.log("GeoRelayDirectory: no local CSV found; entries empty", category: SecureLogger.session, level: .warning) + SecureLogger.log("GeoRelayDirectory: no local CSV found; entries empty", category: .session, level: .warning) return [] } diff --git a/bitchat/Nostr/NostrProtocol.swift b/bitchat/Nostr/NostrProtocol.swift index c2491194..386b89c5 100644 --- a/bitchat/Nostr/NostrProtocol.swift +++ b/bitchat/Nostr/NostrProtocol.swift @@ -79,7 +79,7 @@ struct NostrProtocol { // Successfully unwrapped gift wrap } catch { SecureLogger.log("โŒ Failed to unwrap gift wrap: \(error)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) throw error } @@ -93,7 +93,7 @@ struct NostrProtocol { // Successfully opened seal } catch { SecureLogger.log("โŒ Failed to open seal: \(error)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) throw error } diff --git a/bitchat/Nostr/NostrRelayManager.swift b/bitchat/Nostr/NostrRelayManager.swift index 99b363f8..502aad64 100644 --- a/bitchat/Nostr/NostrRelayManager.swift +++ b/bitchat/Nostr/NostrRelayManager.swift @@ -82,10 +82,10 @@ final class NostrRelayManager: ObservableObject { let ready = await TorManager.shared.awaitReady() await MainActor.run { if !ready { - SecureLogger.log("โŒ Tor not ready; aborting relay connections (fail-closed)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Tor not ready; aborting relay connections (fail-closed)", category: .session, level: .error) return } - SecureLogger.log("๐ŸŒ Connecting to \(self.relays.count) Nostr relays (via Tor)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐ŸŒ Connecting to \(self.relays.count) Nostr relays (via Tor)", category: .session, level: .debug) for relay in self.relays { self.connectToRelay(relay.url) } @@ -231,12 +231,12 @@ final class NostrRelayManager: ObservableObject { do { let message = try encoder.encode(req) guard let messageString = String(data: message, encoding: .utf8) else { - SecureLogger.log("โŒ Failed to encode subscription request", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to encode subscription request", category: .session, level: .error) return } // SecureLogger.log("๐Ÿ“‹ Subscription filter JSON: \(messageString.prefix(200))...", - // category: SecureLogger.session, level: .debug) + // category: .session, level: .debug) // Target specific relays if provided; else default. Filter permanently failed relays. let baseUrls = relayUrls ?? Self.defaultRelays @@ -252,7 +252,7 @@ final class NostrRelayManager: ObservableObject { self.pendingSubscriptions[url] = map } SecureLogger.log("๐Ÿ“‹ Queued subscription id=\(id) for \(urls.count) relay(s)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Ensure we actually have sockets opening to these relays so queued REQs can flush ensureConnections(to: urls) // If some targets are already connected, flush immediately for them @@ -263,7 +263,7 @@ final class NostrRelayManager: ObservableObject { } } catch { SecureLogger.log("โŒ Failed to encode subscription request: \(error)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) } } @@ -295,7 +295,7 @@ final class NostrRelayManager: ObservableObject { private func connectToRelay(_ urlString: String) { guard let url = URL(string: urlString) else { - SecureLogger.log("Invalid relay URL: \(urlString)", category: SecureLogger.session, level: .warning) + SecureLogger.log("Invalid relay URL: \(urlString)", category: .session, level: .warning) return } @@ -321,7 +321,7 @@ final class NostrRelayManager: ObservableObject { let ready = await TorManager.shared.awaitReady() await MainActor.run { if ready { self.connectToRelay(urlString) } - else { SecureLogger.log("โŒ Tor not ready; skipping connection to \(urlString)", category: SecureLogger.session, level: .error) } + else { SecureLogger.log("โŒ Tor not ready; skipping connection to \(urlString)", category: .session, level: .error) } } } return @@ -341,13 +341,13 @@ final class NostrRelayManager: ObservableObject { DispatchQueue.main.async { if error == nil { SecureLogger.log("โœ… Connected to Nostr relay: \(urlString)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) self?.updateRelayStatus(urlString, isConnected: true) // Flush any pending subscriptions for this relay self?.flushPendingSubscriptions(for: urlString) } else { SecureLogger.log("โŒ Failed to connect to Nostr relay \(urlString): \(error?.localizedDescription ?? "Unknown error")", - category: SecureLogger.session, level: .error) + category: .session, level: .error) self?.updateRelayStatus(urlString, isConnected: false, error: error) // Trigger disconnection handler for proper backoff self?.handleDisconnection(relayUrl: urlString, error: error ?? NSError(domain: "NostrRelay", code: -1, userInfo: nil)) @@ -365,7 +365,7 @@ final class NostrRelayManager: ObservableObject { connection.send(.string(messageString)) { error in if let error = error { SecureLogger.log("โŒ Failed to send pending subscription to \(relayUrl): \(error)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) } else { Task { @MainActor in var subs = self.subscriptions[relayUrl] ?? Set() @@ -415,7 +415,7 @@ final class NostrRelayManager: ObservableObject { case .event(let subId, let event): if event.kind != 1059 { SecureLogger.log("๐Ÿ“ฅ Event kind=\(event.kind) id=\(event.id.prefix(16))โ€ฆ relay=\(relayUrl)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } if let index = self.relays.firstIndex(where: { $0.url == relayUrl }) { self.relays[index].messagesReceived += 1 @@ -424,7 +424,7 @@ final class NostrRelayManager: ObservableObject { handler(event) } else { SecureLogger.log("โš ๏ธ No handler for subscription \(subId)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } case .eose: // No-op for now @@ -433,11 +433,11 @@ final class NostrRelayManager: ObservableObject { if success { _ = Self.pendingGiftWrapIDs.remove(eventId) SecureLogger.log("โœ… Accepted id=\(eventId.prefix(16))โ€ฆ relay=\(relayUrl)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } else { let isGiftWrap = Self.pendingGiftWrapIDs.remove(eventId) != nil SecureLogger.log("๐Ÿ“ฎ Rejected id=\(eventId.prefix(16))โ€ฆ reason=\(reason)", - category: SecureLogger.session, level: isGiftWrap ? .warning : .error) + category: .session, level: isGiftWrap ? .warning : .error) } case .notice: break @@ -452,16 +452,16 @@ final class NostrRelayManager: ObservableObject { let message = String(data: data, encoding: .utf8) ?? "" SecureLogger.log("๐Ÿ“ค Send kind=\(event.kind) id=\(event.id.prefix(16))โ€ฆ relay=\(relayUrl)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) connection.send(.string(message)) { [weak self] error in DispatchQueue.main.async { if let error = error { SecureLogger.log("โŒ Failed to send event to \(relayUrl): \(error)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) } else { // SecureLogger.log("โœ… Event sent to relay: \(relayUrl)", - // category: SecureLogger.session, level: .debug) + // category: .session, level: .debug) // Update relay stats if let index = self?.relays.firstIndex(where: { $0.url == relayUrl }) { self?.relays[index].messagesSent += 1 @@ -470,7 +470,7 @@ final class NostrRelayManager: ObservableObject { } } } catch { - SecureLogger.log("Failed to encode event: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to encode event: \(error)", category: .session, level: .error) } } @@ -509,7 +509,7 @@ final class NostrRelayManager: ObservableObject { errorDescription.contains("dns") || (ns.domain == NSURLErrorDomain && ns.code == NSURLErrorBadServerResponse) { if relays.first(where: { $0.url == relayUrl })?.lastError == nil { - SecureLogger.log("Nostr relay permanent failure for \(relayUrl) - not retrying (code=\(ns.code))", category: SecureLogger.session, level: .warning) + SecureLogger.log("Nostr relay permanent failure for \(relayUrl) - not retrying (code=\(ns.code))", category: .session, level: .warning) } if let index = relays.firstIndex(where: { $0.url == relayUrl }) { relays[index].lastError = error @@ -528,7 +528,7 @@ final class NostrRelayManager: ObservableObject { // Stop attempting after max attempts if relays[index].reconnectAttempts >= maxReconnectAttempts { SecureLogger.log("Max reconnection attempts (\(maxReconnectAttempts)) reached for \(relayUrl)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return } diff --git a/bitchat/Services/BLEService.swift b/bitchat/Services/BLEService.swift index 3564beef..14128a31 100644 --- a/bitchat/Services/BLEService.swift +++ b/bitchat/Services/BLEService.swift @@ -231,7 +231,7 @@ final class BLEService: NSObject { // If single chunk exceeds cap, drop it immediately if newSize > capBytes { SecureLogger.log("โš ๏ธ Dropping oversized write chunk (\(newSize)B) for peripheral \(uuid)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } else { // Append and trim from the front to respect cap var total = queue.reduce(0) { $0 + $1.count } @@ -245,7 +245,7 @@ final class BLEService: NSObject { total -= removed.count } SecureLogger.log("๐Ÿ“‰ Trimmed pending write buffer for \(uuid) by \(removedBytes)B to \(total)B", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } self.pendingPeripheralWrites[uuid] = queue.isEmpty ? nil : queue } @@ -342,7 +342,7 @@ final class BLEService: NSObject { // This ensures we send pending messages only when session is truly established noiseService.onPeerAuthenticated = { [weak self] peerID, fingerprint in SecureLogger.log("๐Ÿ” Noise session authenticated with \(peerID), fingerprint: \(fingerprint.prefix(16))...", - category: SecureLogger.noise, level: .debug) + category: .noise, level: .debug) // Send any messages that were queued during handshake self?.messageQueue.async { [weak self] in self?.sendPendingMessagesAfterHandshake(for: peerID) @@ -588,7 +588,7 @@ final class BLEService: NSObject { func sendFavoriteNotification(to peerID: String, isFavorite: Bool) { SecureLogger.log("๐Ÿ”” sendFavoriteNotification called - peerID: \(peerID), isFavorite: \(isFavorite)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Include Nostr public key in the notification var content = isFavorite ? "[FAVORITED]" : "[UNFAVORITED]" @@ -597,11 +597,11 @@ final class BLEService: NSObject { if let myNostrIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() { content += ":" + myNostrIdentity.npub SecureLogger.log("๐Ÿ“ Sending favorite notification with Nostr npub: \(myNostrIdentity.npub)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } SecureLogger.log("๐Ÿ“ค Sending favorite notification to \(peerID): \(content)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) sendPrivateMessage(content, to: peerID, messageID: UUID().uuidString) } @@ -612,7 +612,7 @@ final class BLEService: NSObject { if noiseService.hasEstablishedSession(with: peerID) { SecureLogger.log("๐Ÿ“ค Sending READ receipt for message \(receipt.originalMessageID) to \(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) do { let encrypted = try noiseService.encrypt(payload, for: peerID) let packet = BitchatPacket( @@ -630,7 +630,7 @@ final class BLEService: NSObject { messageQueue.async { [weak self] in self?.broadcastPacket(packet) } } } catch { - SecureLogger.log("Failed to send read receipt: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to send read receipt: \(error)", category: .noise, level: .error) } } else { // Queue for after handshake and initiate if needed @@ -640,7 +640,7 @@ final class BLEService: NSObject { } if !noiseService.hasSession(with: peerID) { initiateNoiseHandshake(with: peerID) } SecureLogger.log("๐Ÿ•’ Queued READ receipt for \(peerID) until handshake completes", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -682,7 +682,7 @@ final class BLEService: NSObject { messageQueue.async { [weak self] in self?.broadcastPacket(packet) } } } catch { - SecureLogger.log("Failed to send verification payload: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to send verification payload: \(error)", category: .noise, level: .error) } } @@ -748,7 +748,7 @@ final class BLEService: NSObject { guard let self = self else { return } guard content.count <= self.maxMessageLength else { - SecureLogger.log("Message too long: \(content.count) chars", category: SecureLogger.session, level: .error) + SecureLogger.log("Message too long: \(content.count) chars", category: .session, level: .error) return } @@ -771,7 +771,7 @@ final class BLEService: NSObject { ttl: self.messageTTL ) guard let signedPacket = self.noiseService.signPacket(basePacket) else { - SecureLogger.log("โŒ Failed to sign public message", category: SecureLogger.security, level: .error) + SecureLogger.log("โŒ Failed to sign public message", category: .security, level: .error) return } // Pre-mark our own broadcast as processed to avoid handling relayed self copy @@ -789,7 +789,7 @@ final class BLEService: NSObject { // MARK: - Private Message Handling private func sendPrivateMessage(_ content: String, to recipientID: String, messageID: String) { - SecureLogger.log("๐Ÿ“จ Sending PM to \(recipientID): \(content.prefix(30))...", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“จ Sending PM to \(recipientID): \(content.prefix(30))...", category: .session, level: .debug) // Check if we have an established Noise session if noiseService.hasEstablishedSession(with: recipientID) { @@ -798,7 +798,7 @@ final class BLEService: NSObject { // Create TLV-encoded private message let privateMessage = PrivateMessagePacket(messageID: messageID, content: content) guard let tlvData = privateMessage.encode() else { - SecureLogger.log("Failed to encode private message with TLV", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to encode private message with TLV", category: .noise, level: .error) return } @@ -847,11 +847,11 @@ final class BLEService: NSObject { self?.delegate?.didUpdateMessageDeliveryStatus(messageID, status: .sent) } } catch { - SecureLogger.log("Failed to encrypt message: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to encrypt message: \(error)", category: .noise, level: .error) } } else { // Queue message for sending after handshake completes - SecureLogger.log("๐Ÿค No session with \(recipientID), initiating handshake and queueing message", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿค No session with \(recipientID), initiating handshake and queueing message", category: .session, level: .debug) // Queue the message (especially important for favorite notifications) collectionsQueue.sync(flags: .barrier) { @@ -896,7 +896,7 @@ final class BLEService: NSObject { } } } catch { - SecureLogger.log("Failed to initiate handshake: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to initiate handshake: \(error)", category: .noise, level: .error) } } @@ -911,7 +911,7 @@ final class BLEService: NSObject { guard let messages = pendingMessages, !messages.isEmpty else { return } SecureLogger.log("๐Ÿ“ค Sending \(messages.count) pending messages after handshake to \(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Send each pending message directly (we know session is established) for (content, messageID) in messages { @@ -919,7 +919,7 @@ final class BLEService: NSObject { // Use the same TLV format as normal sends to keep receiver decoding consistent let privateMessage = PrivateMessagePacket(messageID: messageID, content: content) guard let tlvData = privateMessage.encode() else { - SecureLogger.log("Failed to encode pending private message TLV", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to encode pending private message TLV", category: .noise, level: .error) continue } @@ -947,10 +947,10 @@ final class BLEService: NSObject { } SecureLogger.log("โœ… Sent pending message \(messageID) to \(peerID) after handshake", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } catch { SecureLogger.log("Failed to send pending message after handshake: \(error)", - category: SecureLogger.noise, level: .error) + category: .noise, level: .error) // Notify delegate of failure notifyUI { [weak self] in @@ -966,7 +966,7 @@ final class BLEService: NSObject { // Encode once using a small per-type padding policy, then delegate by type let padForBLE = padPolicy(for: packet.type) guard let data = packet.toBinaryData(padding: padForBLE) else { - SecureLogger.log("โŒ Failed to convert packet to binary data", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to convert packet to binary data", category: .session, level: .error) return } if packet.type == MessageType.noiseEncrypted.rawValue { @@ -1037,7 +1037,7 @@ final class BLEService: NSObject { guard let self = self else { return } if self.pendingNotifications.count < TransportConfig.blePendingNotificationsCapCount { self.pendingNotifications.append((data: data, centrals: [central])) - SecureLogger.log("๐Ÿ“‹ Queued encrypted packet for retry (notification queue full)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“‹ Queued encrypted packet for retry (notification queue full)", category: .session, level: .debug) } } } @@ -1150,7 +1150,7 @@ final class BLEService: NSObject { if byMsg[msgID] == nil { byMsg[msgID] = (packet: packet, enqueuedAt: Date()) self.pendingDirectedRelays[recipientPeerID] = byMsg - SecureLogger.log("๐Ÿงณ Spooling directed packet for \(recipientPeerID) mid=\(msgID.prefix(8))โ€ฆ", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿงณ Spooling directed packet for \(recipientPeerID) mid=\(msgID.prefix(8))โ€ฆ", category: .session, level: .debug) } } } @@ -1322,7 +1322,7 @@ final class BLEService: NSObject { if let originalPacket = BinaryProtocol.decode(reassembled) { handleReceivedPacket(originalPacket, from: peerID) } else { - SecureLogger.log("โŒ Failed to decode reassembled packet (type=\(originalType), total=\(total))", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to decode reassembled packet (type=\(originalType), total=\(total))", category: .session, level: .error) } // Cleanup @@ -1343,7 +1343,7 @@ final class BLEService: NSObject { if packet.type != MessageType.announce.rawValue { // Log packet details for debugging SecureLogger.log("๐Ÿ“ฆ Handling packet type \(packet.type) from \(senderID), messageID: \(messageID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } // Efficient deduplication @@ -1353,7 +1353,7 @@ final class BLEService: NSObject { // It's normal to see these as duplicates - don't log them to reduce noise if packet.type != MessageType.announce.rawValue { SecureLogger.log("โš ๏ธ Duplicate packet ignored: \(messageID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } // In sparse graphs (<=2 neighbors), keep the pending relay to ensure bridging. // In denser graphs, cancel the pending relay to reduce redundant floods. @@ -1406,7 +1406,7 @@ final class BLEService: NSObject { handleLeave(packet, from: senderID) default: - SecureLogger.log("โš ๏ธ Unknown message type: \(packet.type)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Unknown message type: \(packet.type)", category: .session, level: .warning) break } @@ -1446,7 +1446,7 @@ final class BLEService: NSObject { private func handleAnnounce(_ packet: BitchatPacket, from peerID: String) { guard let announcement = AnnouncementPacket.decode(from: packet.payload) else { - SecureLogger.log("โŒ Failed to decode announce packet from \(peerID)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to decode announce packet from \(peerID)", category: .session, level: .error) return } @@ -1454,7 +1454,7 @@ final class BLEService: NSObject { // This helps detect relayed or spoofed announces. Only warn in release; assert in debug. let derivedFromKey = PeerIDUtils.derivePeerID(fromPublicKey: announcement.noisePublicKey) if derivedFromKey != peerID { - SecureLogger.log("โš ๏ธ Announce sender mismatch: derived \(derivedFromKey.prefix(8))โ€ฆ vs packet \(peerID.prefix(8))โ€ฆ", category: SecureLogger.security, level: .warning) + SecureLogger.log("โš ๏ธ Announce sender mismatch: derived \(derivedFromKey.prefix(8))โ€ฆ vs packet \(peerID.prefix(8))โ€ฆ", category: .security, level: .warning) } @@ -1471,11 +1471,11 @@ final class BLEService: NSObject { if packet.signature != nil { verifiedAnnounce = noiseService.verifyPacketSignature(packet, publicKey: announcement.signingPublicKey) if !verifiedAnnounce { - SecureLogger.log("โš ๏ธ Signature verification for announce failed \(peerID.prefix(8))", category: SecureLogger.security, level: .warning) + SecureLogger.log("โš ๏ธ Signature verification for announce failed \(peerID.prefix(8))", category: .security, level: .warning) } } if let existingKey = existingPeerForVerify?.noisePublicKey, existingKey != announcement.noisePublicKey { - SecureLogger.log("โš ๏ธ Announce key mismatch for \(peerID.prefix(8))โ€ฆ โ€” keeping unverified", category: SecureLogger.security, level: .warning) + SecureLogger.log("โš ๏ธ Announce key mismatch for \(peerID.prefix(8))โ€ฆ โ€” keeping unverified", category: .security, level: .warning) verifiedAnnounce = false } @@ -1508,7 +1508,7 @@ final class BLEService: NSObject { // Require verified announce; ignore otherwise (no backward compatibility) if !verified { - SecureLogger.log("โŒ Ignoring unverified announce from \(peerID.prefix(8))โ€ฆ", category: SecureLogger.security, level: .warning) + SecureLogger.log("โŒ Ignoring unverified announce from \(peerID.prefix(8))โ€ฆ", category: .security, level: .warning) return } @@ -1541,17 +1541,17 @@ final class BLEService: NSObject { if isDirectAnnounce || hasPeripheralConnection || hasCentralSubscription { let now = Date() if existingPeer == nil { - SecureLogger.log("๐Ÿ†• New peer: \(announcement.nickname)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ†• New peer: \(announcement.nickname)", category: .session, level: .debug) } else if wasDisconnected { // Debounce 'reconnected' logs within short window if let last = lastReconnectLogAt[peerID], now.timeIntervalSince(last) < TransportConfig.bleReconnectLogDebounceSeconds { // Skip duplicate log } else { - SecureLogger.log("๐Ÿ”„ Peer \(announcement.nickname) reconnected", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ”„ Peer \(announcement.nickname) reconnected", category: .session, level: .debug) lastReconnectLogAt[peerID] = now } } else if existingPeer?.nickname != announcement.nickname { - SecureLogger.log("๐Ÿ”„ Peer \(peerID) changed nickname: \(existingPeer?.nickname ?? "Unknown") -> \(announcement.nickname)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ”„ Peer \(peerID) changed nickname: \(existingPeer?.nickname ?? "Unknown") -> \(announcement.nickname)", category: .session, level: .debug) } } } @@ -1661,12 +1661,12 @@ final class BLEService: NSObject { } guard accepted else { - SecureLogger.log("๐Ÿšซ Dropping public message from unverified or unknown peer \(peerID.prefix(8))โ€ฆ", category: SecureLogger.security, level: .warning) + SecureLogger.log("๐Ÿšซ Dropping public message from unverified or unknown peer \(peerID.prefix(8))โ€ฆ", category: .security, level: .warning) return } guard let content = String(data: packet.payload, encoding: .utf8) else { - SecureLogger.log("โŒ Failed to decode message payload as UTF-8", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to decode message payload as UTF-8", category: .session, level: .error) return } // Determine if we have a direct link to the sender @@ -1678,7 +1678,7 @@ final class BLEService: NSObject { } let pathTag = hasDirectLink ? "direct" : "mesh" - SecureLogger.log("๐Ÿ’ฌ [\(senderNickname)] TTL:\(packet.ttl) (\(pathTag)): \(String(content.prefix(50)))\(content.count > 50 ? "..." : "")", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ’ฌ [\(senderNickname)] TTL:\(packet.ttl) (\(pathTag)): \(String(content.prefix(50)))\(content.count > 50 ? "..." : "")", category: .session, level: .debug) let ts = Date(timeIntervalSince1970: Double(packet.timestamp) / 1000) notifyUI { [weak self] in @@ -1710,7 +1710,7 @@ final class BLEService: NSObject { // Session establishment will trigger onPeerAuthenticated callback // which will send any pending messages at the right time } catch { - SecureLogger.log("Failed to process handshake: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to process handshake: \(error)", category: .noise, level: .error) // Try initiating a new handshake if !noiseService.hasSession(with: peerID) { initiateNoiseHandshake(with: peerID) @@ -1721,16 +1721,16 @@ final class BLEService: NSObject { private func handleNoiseEncrypted(_ packet: BitchatPacket, from peerID: String) { SecureLogger.log("๐Ÿ” handleNoiseEncrypted called for packet from \(peerID)", - category: SecureLogger.noise, level: .debug) + category: .noise, level: .debug) guard let recipientID = packet.recipientID else { - SecureLogger.log("โš ๏ธ Encrypted message has no recipient ID", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Encrypted message has no recipient ID", category: .session, level: .warning) return } let recipientHex = recipientID.hexEncodedString() if recipientHex != myPeerID { - SecureLogger.log("๐Ÿ” Encrypted message not for me (for \(recipientHex), I am \(myPeerID))", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ” Encrypted message not for me (for \(recipientHex), I am \(myPeerID))", category: .session, level: .debug) return } @@ -1772,19 +1772,19 @@ final class BLEService: NSObject { self?.delegate?.didReceiveNoisePayload(from: peerID, type: .verifyResponse, payload: Data(payloadData), timestamp: ts) } default: - SecureLogger.log("โš ๏ธ Unknown noise payload type: \(payloadType)", category: SecureLogger.noise, level: .warning) + SecureLogger.log("โš ๏ธ Unknown noise payload type: \(payloadType)", category: .noise, level: .warning) } } catch NoiseEncryptionError.sessionNotEstablished { // We received an encrypted message before establishing a session with this peer. // Trigger a handshake so future messages can be decrypted. SecureLogger.log("๐Ÿ”‘ Encrypted message from \(peerID) without session; initiating handshake", - category: SecureLogger.noise, level: .debug) + category: .noise, level: .debug) if !noiseService.hasSession(with: peerID) { initiateNoiseHandshake(with: peerID) } } catch { SecureLogger.log("โŒ Failed to decrypt message from \(peerID): \(error)", - category: SecureLogger.noise, level: .error) + category: .noise, level: .error) } } @@ -1808,7 +1808,7 @@ final class BLEService: NSObject { // MARK: - Helper Functions private func sendLeave() { - SecureLogger.log("๐Ÿ‘‹ Sending leave announcement", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ‘‹ Sending leave announcement", category: .session, level: .debug) let packet = BitchatPacket( type: MessageType.leave.rawValue, ttl: messageTTL, @@ -1845,7 +1845,7 @@ final class BLEService: NSObject { ) guard let payload = announcement.encode() else { - SecureLogger.log("โŒ Failed to encode announce packet", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to encode announce packet", category: .session, level: .error) return } @@ -1862,7 +1862,7 @@ final class BLEService: NSObject { // Sign the packet using the noise private key guard let signedPacket = noiseService.signPacket(packet) else { - SecureLogger.log("โŒ Failed to sign announce packet", category: SecureLogger.security, level: .error) + SecureLogger.log("โŒ Failed to sign announce packet", category: .security, level: .error) return } @@ -1895,7 +1895,7 @@ final class BLEService: NSObject { ) broadcastPacket(packet) } catch { - SecureLogger.log("Failed to send delivery ACK: \(error)", category: SecureLogger.noise, level: .error) + SecureLogger.log("Failed to send delivery ACK: \(error)", category: .noise, level: .error) } } else { // Queue for after handshake and initiate if needed @@ -1905,7 +1905,7 @@ final class BLEService: NSObject { } if !noiseService.hasSession(with: peerID) { initiateNoiseHandshake(with: peerID) } SecureLogger.log("๐Ÿ•’ Queued DELIVERED ack for \(peerID) until handshake completes", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -1917,7 +1917,7 @@ final class BLEService: NSObject { } guard !payloads.isEmpty else { return } SecureLogger.log("๐Ÿ“ค Sending \(payloads.count) pending noise payloads to \(peerID) after handshake", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) for payload in payloads { do { let encrypted = try noiseService.encrypt(payload, for: peerID) @@ -1933,7 +1933,7 @@ final class BLEService: NSObject { broadcastPacket(packet) } catch { SecureLogger.log("โŒ Failed to send pending noise payload to \(peerID): \(error)", - category: SecureLogger.noise, level: .error) + category: .noise, level: .error) } } } @@ -2104,7 +2104,7 @@ final class BLEService: NSObject { if !peer.isConnected { if age > retention { SecureLogger.log("๐Ÿ—‘๏ธ Removing stale peer after reachability window: \(peerID) (\(peer.nickname))", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) peers.removeValue(forKey: peerID) removedOfflineCount += 1 } @@ -2396,7 +2396,7 @@ extension BLEService: CBCentralManagerDelegate { // Connect to the peripheral with options for faster connection SecureLogger.log("๐Ÿ“ฑ Connect: \(advertisedName) [RSSI:\(rssiValue)]", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Use connection options for faster reconnection let options: [String: Any] = [ @@ -2416,7 +2416,7 @@ extension BLEService: CBCentralManagerDelegate { // Connection timed out - cancel it SecureLogger.log("โฑ๏ธ Timeout: \(advertisedName)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) central.cancelPeripheralConnection(peripheral) self.peripherals[peripheralID] = nil self.recentConnectTimeouts[peripheralID] = Date() @@ -2449,7 +2449,7 @@ func centralManager(_ central: CBCentralManager, didConnect peripheral: CBPeriph failureCounts[peripheralID] = 0 recentConnectTimeouts.removeValue(forKey: peripheralID) - SecureLogger.log("โœ… Connected: \(peripheral.name ?? "Unknown") [\(peripheralID)]", category: SecureLogger.session, level: .debug) + SecureLogger.log("โœ… Connected: \(peripheral.name ?? "Unknown") [\(peripheralID)]", category: .session, level: .debug) // Discover services peripheral.discoverServices([BLEService.serviceUUID]) @@ -2462,7 +2462,7 @@ func centralManager(_ central: CBCentralManager, didConnect peripheral: CBPeriph let peerID = peripherals[peripheralID]?.peerID SecureLogger.log("๐Ÿ“ฑ Disconnect: \(peerID ?? peripheralID)\(error != nil ? " (\(error!.localizedDescription))" : "")", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // If disconnect carried an error (often timeout), apply short backoff to avoid thrash if error != nil { @@ -2517,7 +2517,7 @@ func centralManager(_ central: CBCentralManager, didConnect peripheral: CBPeriph // Clean up the references peripherals.removeValue(forKey: peripheralID) - SecureLogger.log("โŒ Failed to connect to peripheral: \(peripheral.name ?? "Unknown") [\(peripheralID)] - Error: \(error?.localizedDescription ?? "Unknown")", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to connect to peripheral: \(peripheral.name ?? "Unknown") [\(peripheralID)] - Error: \(error?.localizedDescription ?? "Unknown")", category: .session, level: .error) failureCounts[peripheralID, default: 0] += 1 // Try next candidate bleQueue.async { [weak self] in self?.tryConnectFromQueue() } @@ -2590,7 +2590,7 @@ extension BLEService { ] central.connect(peripheral, options: options) lastGlobalConnectAttempt = Date() - SecureLogger.log("โฉ Queue connect: \(candidate.name) [RSSI:\(candidate.rssi)]", category: SecureLogger.session, level: .debug) + SecureLogger.log("โฉ Queue connect: \(candidate.name) [RSSI:\(candidate.rssi)]", category: .session, level: .debug) } } @@ -2635,7 +2635,7 @@ extension BLEService { extension BLEService: CBPeripheralDelegate { func peripheral(_ peripheral: CBPeripheral, didDiscoverServices error: Error?) { if let error = error { - SecureLogger.log("โŒ Error discovering services for \(peripheral.name ?? "Unknown"): \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Error discovering services for \(peripheral.name ?? "Unknown"): \(error.localizedDescription)", category: .session, level: .error) // Retry service discovery after a delay DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) { guard peripheral.state == .connected else { return } @@ -2645,7 +2645,7 @@ extension BLEService: CBPeripheralDelegate { } guard let services = peripheral.services else { - SecureLogger.log("โš ๏ธ No services discovered for \(peripheral.name ?? "Unknown")", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ No services discovered for \(peripheral.name ?? "Unknown")", category: .session, level: .warning) return } @@ -2661,12 +2661,12 @@ extension BLEService: CBPeripheralDelegate { func peripheral(_ peripheral: CBPeripheral, didDiscoverCharacteristicsFor service: CBService, error: Error?) { if let error = error { - SecureLogger.log("โŒ Error discovering characteristics for \(peripheral.name ?? "Unknown"): \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Error discovering characteristics for \(peripheral.name ?? "Unknown"): \(error.localizedDescription)", category: .session, level: .error) return } guard let characteristic = service.characteristics?.first(where: { $0.uuid == BLEService.characteristicUUID }) else { - SecureLogger.log("โš ๏ธ No matching characteristic found for \(peripheral.name ?? "Unknown")", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ No matching characteristic found for \(peripheral.name ?? "Unknown")", category: .session, level: .warning) return } @@ -2683,7 +2683,7 @@ extension BLEService: CBPeripheralDelegate { // Verify characteristic supports reliable writes if !characteristic.properties.contains(.write) { - SecureLogger.log("โš ๏ธ Characteristic doesn't support reliable writes (withResponse)!", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Characteristic doesn't support reliable writes (withResponse)!", category: .session, level: .warning) } // Store characteristic in our consolidated structure @@ -2696,7 +2696,7 @@ extension BLEService: CBPeripheralDelegate { // Subscribe for notifications if characteristic.properties.contains(.notify) { peripheral.setNotifyValue(true, for: characteristic) - SecureLogger.log("๐Ÿ”” Subscribed to notifications from \(peripheral.name ?? "Unknown")", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ”” Subscribed to notifications from \(peripheral.name ?? "Unknown")", category: .session, level: .debug) // Send announce after subscription is confirmed (force send for new connection) messageQueue.asyncAfter(deadline: .now() + TransportConfig.blePostSubscribeAnnounceDelaySeconds) { [weak self] in @@ -2707,18 +2707,18 @@ extension BLEService: CBPeripheralDelegate { self?.rebroadcastRecentAnnounces() } } else { - SecureLogger.log("โš ๏ธ Characteristic does not support notifications", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Characteristic does not support notifications", category: .session, level: .warning) } } func peripheral(_ peripheral: CBPeripheral, didUpdateValueFor characteristic: CBCharacteristic, error: Error?) { if let error = error { - SecureLogger.log("โŒ Error receiving notification: \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Error receiving notification: \(error.localizedDescription)", category: .session, level: .error) return } guard let data = characteristic.value else { - SecureLogger.log("โš ๏ธ No data in notification", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ No data in notification", category: .session, level: .warning) return } @@ -2729,7 +2729,7 @@ extension BLEService: CBPeripheralDelegate { // Avoid dumping entire payload; log size and short prefix for diagnostics let prefix = data.prefix(16).map { String(format: "%02x", $0) }.joined(separator: " ") SecureLogger.log("โŒ Failed to decode notification packet (len=\(data.count), prefix=\(prefix))", - category: SecureLogger.session, level: .error) + category: .session, level: .error) return } @@ -2737,7 +2737,7 @@ extension BLEService: CBPeripheralDelegate { let senderID = packet.senderID.hexEncodedString() // Only log non-announce packets if packet.type != MessageType.announce.rawValue { - SecureLogger.log("๐Ÿ“ฆ Decoded notification packet type: \(packet.type) from sender: \(senderID)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฆ Decoded notification packet type: \(packet.type) from sender: \(senderID)", category: .session, level: .debug) } let peripheralUUID = peripheral.identifier.uuidString @@ -2775,10 +2775,10 @@ extension BLEService: CBPeripheralDelegate { func peripheral(_ peripheral: CBPeripheral, didWriteValueFor characteristic: CBCharacteristic, error: Error?) { if let error = error { - SecureLogger.log("โŒ Write failed to \(peripheral.name ?? peripheral.identifier.uuidString): \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Write failed to \(peripheral.name ?? peripheral.identifier.uuidString): \(error.localizedDescription)", category: .session, level: .error) // Don't retry - just log the error } else { - SecureLogger.log("โœ… Write confirmed to \(peripheral.name ?? peripheral.identifier.uuidString)", category: SecureLogger.session, level: .debug) + SecureLogger.log("โœ… Write confirmed to \(peripheral.name ?? peripheral.identifier.uuidString)", category: .session, level: .debug) } } @@ -2788,14 +2788,14 @@ extension BLEService: CBPeripheralDelegate { } func peripheral(_ peripheral: CBPeripheral, didModifyServices invalidatedServices: [CBService]) { - SecureLogger.log("โš ๏ธ Services modified for \(peripheral.name ?? peripheral.identifier.uuidString)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Services modified for \(peripheral.name ?? peripheral.identifier.uuidString)", category: .session, level: .warning) // Check if our service was invalidated (peer app quit) let hasOurService = peripheral.services?.contains { $0.uuid == BLEService.serviceUUID } ?? false if !hasOurService { // Service is gone - disconnect - SecureLogger.log("โŒ BitChat service removed - disconnecting from \(peripheral.name ?? peripheral.identifier.uuidString)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โŒ BitChat service removed - disconnecting from \(peripheral.name ?? peripheral.identifier.uuidString)", category: .session, level: .warning) centralManager?.cancelPeripheralConnection(peripheral) } else { // Try to rediscover @@ -2805,9 +2805,9 @@ extension BLEService: CBPeripheralDelegate { func peripheral(_ peripheral: CBPeripheral, didUpdateNotificationStateFor characteristic: CBCharacteristic, error: Error?) { if let error = error { - SecureLogger.log("โŒ Error updating notification state: \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Error updating notification state: \(error.localizedDescription)", category: .session, level: .error) } else { - SecureLogger.log("๐Ÿ”” Notification state updated for \(peripheral.name ?? peripheral.identifier.uuidString): \(characteristic.isNotifying ? "ON" : "OFF")", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ”” Notification state updated for \(peripheral.name ?? peripheral.identifier.uuidString): \(characteristic.isNotifying ? "ON" : "OFF")", category: .session, level: .debug) // If notifications are now on, send an announce to ensure this peer knows about us if characteristic.isNotifying { @@ -2822,7 +2822,7 @@ extension BLEService: CBPeripheralDelegate { extension BLEService: CBPeripheralManagerDelegate { func peripheralManagerDidUpdateState(_ peripheral: CBPeripheralManager) { - SecureLogger.log("๐Ÿ“ก Peripheral manager state: \(peripheral.state.rawValue)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ก Peripheral manager state: \(peripheral.state.rawValue)", category: .session, level: .debug) if peripheral.state == .poweredOn { // Remove all services first to ensure clean state @@ -2841,28 +2841,28 @@ extension BLEService: CBPeripheralManagerDelegate { service.characteristics = [characteristic!] // Add service (advertising will start in didAdd delegate) - SecureLogger.log("๐Ÿ”ง Adding BLE service...", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ”ง Adding BLE service...", category: .session, level: .debug) peripheral.add(service) } } func peripheralManager(_ peripheral: CBPeripheralManager, didAdd service: CBService, error: Error?) { if let error = error { - SecureLogger.log("โŒ Failed to add service: \(error.localizedDescription)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to add service: \(error.localizedDescription)", category: .session, level: .error) return } - SecureLogger.log("โœ… Service added successfully, starting advertising", category: SecureLogger.session, level: .debug) + SecureLogger.log("โœ… Service added successfully, starting advertising", category: .session, level: .debug) // Start advertising after service is confirmed added let adData = buildAdvertisementData() peripheral.startAdvertising(adData) - SecureLogger.log("๐Ÿ“ก Started advertising (LocalName: \((adData[CBAdvertisementDataLocalNameKey] as? String) != nil ? "on" : "off"), ID: \(myPeerID.prefix(8))โ€ฆ)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ก Started advertising (LocalName: \((adData[CBAdvertisementDataLocalNameKey] as? String) != nil ? "on" : "off"), ID: \(myPeerID.prefix(8))โ€ฆ)", category: .session, level: .debug) } func peripheralManager(_ peripheral: CBPeripheralManager, central: CBCentral, didSubscribeTo characteristic: CBCharacteristic) { - SecureLogger.log("๐Ÿ“ฅ Central subscribed: \(central.identifier.uuidString)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฅ Central subscribed: \(central.identifier.uuidString)", category: .session, level: .debug) subscribedCentrals.append(central) // Send announce to the newly subscribed central after a small delay to avoid overwhelming messageQueue.asyncAfter(deadline: .now() + TransportConfig.blePostAnnounceDelaySeconds) { [weak self] in @@ -2875,12 +2875,12 @@ extension BLEService: CBPeripheralManagerDelegate { } func peripheralManager(_ peripheral: CBPeripheralManager, central: CBCentral, didUnsubscribeFrom characteristic: CBCharacteristic) { - SecureLogger.log("๐Ÿ“ค Central unsubscribed: \(central.identifier.uuidString)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ค Central unsubscribed: \(central.identifier.uuidString)", category: .session, level: .debug) subscribedCentrals.removeAll { $0.identifier == central.identifier } // Ensure we're still advertising for other devices to find us if peripheral.isAdvertising == false { - SecureLogger.log("๐Ÿ“ก Restarting advertising after central unsubscribed", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ก Restarting advertising after central unsubscribed", category: .session, level: .debug) peripheral.startAdvertising(buildAdvertisementData()) } @@ -2914,7 +2914,7 @@ extension BLEService: CBPeripheralManagerDelegate { } func peripheralManagerIsReady(toUpdateSubscribers peripheral: CBPeripheralManager) { - SecureLogger.log("๐Ÿ“ค Peripheral manager ready to send more notifications", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ค Peripheral manager ready to send more notifications", category: .session, level: .debug) // Retry pending notifications now that queue has space collectionsQueue.async(flags: .barrier) { [weak self] in @@ -2934,11 +2934,11 @@ extension BLEService: CBPeripheralManagerDelegate { // Still full, re-queue self.pendingNotifications.append((data: data, centrals: centrals)) SecureLogger.log("โš ๏ธ Notification queue still full, re-queuing", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) break // Stop trying, wait for next ready callback } else { SecureLogger.log("โœ… Sent pending notification from retry queue", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } else { // Broadcast to all @@ -2953,7 +2953,7 @@ extension BLEService: CBPeripheralManagerDelegate { if !self.pendingNotifications.isEmpty { SecureLogger.log("๐Ÿ“‹ Still have \(self.pendingNotifications.count) pending notifications", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } } @@ -2961,7 +2961,7 @@ extension BLEService: CBPeripheralManagerDelegate { func peripheralManager(_ peripheral: CBPeripheralManager, didReceiveWrite requests: [CBATTRequest]) { // Suppress logs for single write requests to reduce noise if requests.count > 1 { - SecureLogger.log("๐Ÿ“ฅ Received \(requests.count) write requests from central", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฅ Received \(requests.count) write requests from central", category: .session, level: .debug) } // IMPORTANT: Respond immediately to prevent timeouts! @@ -3000,7 +3000,7 @@ extension BLEService: CBPeripheralManagerDelegate { if combined.count >= 2 { let peekType = combined[1] if peekType != MessageType.announce.rawValue { - SecureLogger.log("๐Ÿ“ฅ Accumulated write from central \(centralUUID): size=\(combined.count) (+\(appendedBytes)) bytes (type=\(peekType)), offsets=\(offsets)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฅ Accumulated write from central \(centralUUID): size=\(combined.count) (+\(appendedBytes)) bytes (type=\(peekType)), offsets=\(offsets)", category: .session, level: .debug) } } @@ -3010,7 +3010,7 @@ extension BLEService: CBPeripheralManagerDelegate { pendingWriteBuffers.removeValue(forKey: centralUUID) let senderID = packet.senderID.hexEncodedString() if packet.type != MessageType.announce.rawValue { - SecureLogger.log("๐Ÿ“ฆ Decoded (combined) packet type: \(packet.type) from sender: \(senderID)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฆ Decoded (combined) packet type: \(packet.type) from sender: \(senderID)", category: .session, level: .debug) } if !subscribedCentrals.contains(sorted[0].central) { subscribedCentrals.append(sorted[0].central) @@ -3035,12 +3035,12 @@ extension BLEService: CBPeripheralManagerDelegate { // If buffer grows suspiciously large, reset to avoid memory leak if combined.count > TransportConfig.blePendingWriteBufferCapBytes { // cap for safety pendingWriteBuffers.removeValue(forKey: centralUUID) - SecureLogger.log("โš ๏ธ Dropping oversized pending write buffer (\(combined.count) bytes) for central \(centralUUID)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Dropping oversized pending write buffer (\(combined.count) bytes) for central \(centralUUID)", category: .session, level: .warning) } // If this was a single short write and still failed, log the raw chunk for debugging if !hasMultiple, let only = sorted.first, let raw = only.value { let prefix = raw.prefix(16).map { String(format: "%02x", $0) }.joined(separator: " ") - SecureLogger.log("โŒ Failed to decode packet from central (len=\(raw.count), prefix=\(prefix))", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to decode packet from central (len=\(raw.count), prefix=\(prefix))", category: .session, level: .error) } } } diff --git a/bitchat/Services/FavoritesPersistenceService.swift b/bitchat/Services/FavoritesPersistenceService.swift index 3537c35e..a8b936e0 100644 --- a/bitchat/Services/FavoritesPersistenceService.swift +++ b/bitchat/Services/FavoritesPersistenceService.swift @@ -48,7 +48,7 @@ final class FavoritesPersistenceService: ObservableObject { peerNickname: String ) { SecureLogger.log("โญ๏ธ Adding favorite: \(peerNickname) (\(peerNoisePublicKey.hexEncodedString()))", - category: SecureLogger.session, level: .info) + category: .session, level: .info) let existing = favorites[peerNoisePublicKey] @@ -65,7 +65,7 @@ final class FavoritesPersistenceService: ObservableObject { // Log if this creates a mutual favorite if relationship.isMutual { SecureLogger.log("๐Ÿ’• Mutual favorite relationship established with \(peerNickname)!", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } favorites[peerNoisePublicKey] = relationship @@ -84,7 +84,7 @@ final class FavoritesPersistenceService: ObservableObject { guard let existing = favorites[peerNoisePublicKey] else { return } SecureLogger.log("โญ๏ธ Removing favorite: \(existing.peerNickname) (\(peerNoisePublicKey.hexEncodedString()))", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // If they still favorite us, keep the record but mark us as not favoriting if existing.theyFavoritedUs { @@ -126,7 +126,7 @@ final class FavoritesPersistenceService: ObservableObject { let displayName = peerNickname ?? existing?.peerNickname ?? "Unknown" SecureLogger.log("๐Ÿ“จ Received favorite notification: \(displayName) \(favorited ? "favorited" : "unfavorited") us", - category: SecureLogger.session, level: .info) + category: .session, level: .info) let relationship = FavoriteRelationship( peerNoisePublicKey: peerNoisePublicKey, @@ -148,7 +148,7 @@ final class FavoritesPersistenceService: ObservableObject { // Check if this creates a mutual favorite if relationship.isMutual { SecureLogger.log("๐Ÿ’• Mutual favorite relationship established with \(displayName)!", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } @@ -241,14 +241,14 @@ final class FavoritesPersistenceService: ObservableObject { func updateNoisePublicKey(from oldKey: Data, to newKey: Data, peerNickname: String) { guard let existing = favorites[oldKey] else { SecureLogger.log("โš ๏ธ Cannot update noise key - no favorite found for \(oldKey.hexEncodedString())", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return } // Check if we already have a favorite with the new key if favorites[newKey] != nil { SecureLogger.log("โš ๏ธ Favorite already exists with new key \(newKey.hexEncodedString()), removing old entry", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) favorites.removeValue(forKey: oldKey) saveFavorites() return @@ -302,7 +302,7 @@ final class FavoritesPersistenceService: ObservableObject { /// Clear all favorites - used for panic mode func clearAllFavorites() { - SecureLogger.log("๐Ÿงน Clearing all favorites (panic mode)", category: SecureLogger.session, level: .warning) + SecureLogger.log("๐Ÿงน Clearing all favorites (panic mode)", category: .session, level: .warning) favorites.removeAll() saveFavorites() @@ -336,7 +336,7 @@ final class FavoritesPersistenceService: ObservableObject { // Successfully saved favorites } catch { - SecureLogger.log("Failed to save favorites: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to save favorites: \(error)", category: .session, level: .error) } } @@ -355,13 +355,13 @@ final class FavoritesPersistenceService: ObservableObject { let relationships = try decoder.decode([FavoriteRelationship].self, from: data) SecureLogger.log("โœ… Loaded \(relationships.count) favorite relationships", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // Log Nostr public key info for relationship in relationships { if relationship.peerNostrPublicKey == nil { SecureLogger.log("โš ๏ธ No Nostr public key stored for '\(relationship.peerNickname)'", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } @@ -373,7 +373,7 @@ final class FavoritesPersistenceService: ObservableObject { // Check for duplicates by public key (the actual unique identifier) if let existing = seenPublicKeys[relationship.peerNoisePublicKey] { SecureLogger.log("โš ๏ธ Duplicate favorite found for public key \(relationship.peerNoisePublicKey.hexEncodedString()) - nicknames: '\(existing.peerNickname)' vs '\(relationship.peerNickname)'", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) // Keep the most recent or most complete relationship if relationship.lastUpdated > existing.lastUpdated || @@ -414,7 +414,7 @@ final class FavoritesPersistenceService: ObservableObject { // Log loaded relationships // Loaded relationships successfully } catch { - SecureLogger.log("Failed to load favorites: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to load favorites: \(error)", category: .session, level: .error) } } } diff --git a/bitchat/Services/KeychainManager.swift b/bitchat/Services/KeychainManager.swift index 5d19db41..0ede75e3 100644 --- a/bitchat/Services/KeychainManager.swift +++ b/bitchat/Services/KeychainManager.swift @@ -113,9 +113,9 @@ final class KeychainManager { if status == errSecSuccess { return true } if status == -34018 && !triedWithoutGroup { - SecureLogger.logError(NSError(domain: "Keychain", code: -34018), context: "Missing keychain entitlement", category: SecureLogger.keychain) + SecureLogger.logError(NSError(domain: "Keychain", code: -34018), context: "Missing keychain entitlement", category: .keychain) } else if status != errSecDuplicateItem { - SecureLogger.logError(NSError(domain: "Keychain", code: Int(status)), context: "Error saving to keychain", category: SecureLogger.keychain) + SecureLogger.logError(NSError(domain: "Keychain", code: Int(status)), context: "Error saving to keychain", category: .keychain) } return false } @@ -151,7 +151,7 @@ final class KeychainManager { if status == errSecSuccess { return result as? Data } if status == -34018 { - SecureLogger.logError(NSError(domain: "Keychain", code: -34018), context: "Missing keychain entitlement", category: SecureLogger.keychain) + SecureLogger.logError(NSError(domain: "Keychain", code: -34018), context: "Missing keychain entitlement", category: .keychain) } return nil } @@ -198,7 +198,7 @@ final class KeychainManager { // Delete ALL keychain data for panic mode func deleteAllKeychainData() -> Bool { - SecureLogger.log("Panic mode - deleting all keychain data", category: SecureLogger.security, level: .warning) + SecureLogger.log("Panic mode - deleting all keychain data", category: .security, level: .warning) var totalDeleted = 0 @@ -261,7 +261,7 @@ final class KeychainManager { let deleteStatus = SecItemDelete(deleteQuery as CFDictionary) if deleteStatus == errSecSuccess { totalDeleted += 1 - SecureLogger.log("Deleted keychain item: \(account) from \(service)", category: SecureLogger.keychain, level: .info) + SecureLogger.log("Deleted keychain item: \(account) from \(service)", category: .keychain, level: .info) } } } @@ -303,7 +303,7 @@ final class KeychainManager { totalDeleted += 1 } - SecureLogger.log("Panic mode cleanup completed. Total items deleted: \(totalDeleted)", category: SecureLogger.keychain, level: .warning) + SecureLogger.log("Panic mode cleanup completed. Total items deleted: \(totalDeleted)", category: .keychain, level: .warning) return totalDeleted > 0 } diff --git a/bitchat/Services/LocationChannelManager.swift b/bitchat/Services/LocationChannelManager.swift index 99ea62b7..8cfb4045 100644 --- a/bitchat/Services/LocationChannelManager.swift +++ b/bitchat/Services/LocationChannelManager.swift @@ -198,7 +198,7 @@ final class LocationChannelManager: NSObject, CLLocationManagerDelegate, Observa func locationManager(_ manager: CLLocationManager, didFailWithError error: Error) { // Surface as denied/restricted if relevant; otherwise keep previous state SecureLogger.log("LocationChannelManager: location error: \(error.localizedDescription)", - category: SecureLogger.session, level: .error) + category: .session, level: .error) } // MARK: - Helpers diff --git a/bitchat/Services/MessageRouter.swift b/bitchat/Services/MessageRouter.swift index cc0472e5..460d3bad 100644 --- a/bitchat/Services/MessageRouter.swift +++ b/bitchat/Services/MessageRouter.swift @@ -40,19 +40,19 @@ final class MessageRouter { let reachableMesh = mesh.isPeerReachable(peerID) if reachableMesh { SecureLogger.log("Routing PM via mesh (reachable) to \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // BLEService will initiate a handshake if needed and queue the message mesh.sendPrivateMessage(content, to: peerID, recipientNickname: recipientNickname, messageID: messageID) } else if canSendViaNostr(peerID: peerID) { SecureLogger.log("Routing PM via Nostr to \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) nostr.sendPrivateMessage(content, to: peerID, recipientNickname: recipientNickname, messageID: messageID) } else { // Queue for later (when mesh connects or Nostr mapping appears) if outbox[peerID] == nil { outbox[peerID] = [] } outbox[peerID]?.append((content, recipientNickname, messageID)) SecureLogger.log("Queued PM for \(peerID.prefix(8))โ€ฆ (no mesh, no Nostr mapping) id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -60,11 +60,11 @@ final class MessageRouter { // Prefer mesh for reachable peers; BLE will queue if handshake is needed if mesh.isPeerReachable(peerID) { SecureLogger.log("Routing READ ack via mesh (reachable) to \(peerID.prefix(8))โ€ฆ id=\(receipt.originalMessageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) mesh.sendReadReceipt(receipt, to: peerID) } else { SecureLogger.log("Routing READ ack via Nostr to \(peerID.prefix(8))โ€ฆ id=\(receipt.originalMessageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) nostr.sendReadReceipt(receipt, to: peerID) } } @@ -72,7 +72,7 @@ final class MessageRouter { func sendDeliveryAck(_ messageID: String, to peerID: String) { if mesh.isPeerReachable(peerID) { SecureLogger.log("Routing DELIVERED ack via mesh (reachable) to \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) mesh.sendDeliveryAck(for: messageID, to: peerID) } else { nostr.sendDeliveryAck(for: messageID, to: peerID) @@ -109,17 +109,17 @@ final class MessageRouter { func flushOutbox(for peerID: String) { guard let queued = outbox[peerID], !queued.isEmpty else { return } SecureLogger.log("Flushing outbox for \(peerID.prefix(8))โ€ฆ count=\(queued.count)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) var remaining: [(content: String, nickname: String, messageID: String)] = [] // Prefer mesh if connected; else try Nostr if mapping exists for (content, nickname, messageID) in queued { if mesh.isPeerReachable(peerID) { SecureLogger.log("Outbox -> mesh for \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) mesh.sendPrivateMessage(content, to: peerID, recipientNickname: nickname, messageID: messageID) } else if canSendViaNostr(peerID: peerID) { SecureLogger.log("Outbox -> Nostr for \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) nostr.sendPrivateMessage(content, to: peerID, recipientNickname: nickname, messageID: messageID) } else { // Keep unsent items queued diff --git a/bitchat/Services/NoiseEncryptionService.swift b/bitchat/Services/NoiseEncryptionService.swift index 797e853f..22331687 100644 --- a/bitchat/Services/NoiseEncryptionService.swift +++ b/bitchat/Services/NoiseEncryptionService.swift @@ -270,7 +270,7 @@ final class NoiseEncryptionService { let deletedStatic = KeychainManager.shared.deleteIdentityKey(forKey: "noiseStaticKey") let deletedSigning = KeychainManager.shared.deleteIdentityKey(forKey: "ed25519SigningKey") SecureLogger.logKeyOperation("delete", keyType: "identity keys", success: deletedStatic && deletedSigning) - SecureLogger.log("Panic mode activated - identity cleared", category: SecureLogger.security, level: .warning) + SecureLogger.log("Panic mode activated - identity cleared", category: .security, level: .warning) // Stop rekey timer stopRekeyTimer() } @@ -281,7 +281,7 @@ final class NoiseEncryptionService { let signature = try signingKey.signature(for: data) return signature } catch { - SecureLogger.logError(error, context: "Failed to sign data", category: SecureLogger.noise) + SecureLogger.logError(error, context: "Failed to sign data", category: .noise) return nil } } @@ -292,7 +292,7 @@ final class NoiseEncryptionService { let signingPublicKey = try Curve25519.Signing.PublicKey(rawRepresentation: publicKey) return signingPublicKey.isValidSignature(signature, for: data) } catch { - SecureLogger.logError(error, context: "Failed to verify signature", category: SecureLogger.noise) + SecureLogger.logError(error, context: "Failed to verify signature", category: .noise) return false } } @@ -573,12 +573,12 @@ final class NoiseEncryptionService { // Attempt to rekey the session do { try sessionManager.initiateRekey(for: peerID) - SecureLogger.log("Key rotation initiated for peer: \(peerID)", category: SecureLogger.security, level: .debug) + SecureLogger.log("Key rotation initiated for peer: \(peerID)", category: .security, level: .debug) // Signal that handshake is needed onHandshakeRequired?(peerID) } catch { - SecureLogger.logError(error, context: "Failed to initiate rekey for peer: \(peerID)", category: SecureLogger.session) + SecureLogger.logError(error, context: "Failed to initiate rekey for peer: \(peerID)", category: .session) } } } diff --git a/bitchat/Services/NostrTransport.swift b/bitchat/Services/NostrTransport.swift index c1fad64d..be9517cc 100644 --- a/bitchat/Services/NostrTransport.swift +++ b/bitchat/Services/NostrTransport.swift @@ -52,30 +52,30 @@ final class NostrTransport: Transport { guard let recipientNpub = resolveRecipientNpub(for: peerID) else { return } guard let senderIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() else { return } SecureLogger.log("NostrTransport: preparing PM to \(recipientNpub.prefix(16))โ€ฆ for peerID \(peerID.prefix(8))โ€ฆ id=\(messageID.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Convert recipient npub -> hex (x-only) let recipientHex: String do { let (hrp, data) = try Bech32.decode(recipientNpub) guard hrp == "npub" else { - SecureLogger.log("NostrTransport: recipient key not npub (hrp=\(hrp))", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: recipient key not npub (hrp=\(hrp))", category: .session, level: .error) return } recipientHex = data.hexEncodedString() } catch { - SecureLogger.log("NostrTransport: failed to decode npub -> hex: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to decode npub -> hex: \(error)", category: .session, level: .error) return } guard let embedded = NostrEmbeddedBitChat.encodePMForNostr(content: content, messageID: messageID, recipientPeerID: peerID, senderPeerID: senderPeerID) else { - SecureLogger.log("NostrTransport: failed to embed PM packet", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to embed PM packet", category: .session, level: .error) return } guard let event = try? NostrProtocol.createPrivateMessage(content: embedded, recipientPubkey: recipientHex, senderIdentity: senderIdentity) else { - SecureLogger.log("NostrTransport: failed to build Nostr event for PM", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to build Nostr event for PM", category: .session, level: .error) return } SecureLogger.log("NostrTransport: sending PM giftWrap id=\(event.id.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) NostrRelayManager.shared.sendEvent(event) } } @@ -100,7 +100,7 @@ final class NostrTransport: Transport { guard let recipientNpub = resolveRecipientNpub(for: item.peerID) else { scheduleNextReadAck(); return } guard let senderIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() else { scheduleNextReadAck(); return } SecureLogger.log("NostrTransport: preparing READ ack for id=\(item.receipt.originalMessageID.prefix(8))โ€ฆ to \(recipientNpub.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Convert recipient npub -> hex let recipientHex: String do { @@ -109,15 +109,15 @@ final class NostrTransport: Transport { recipientHex = data.hexEncodedString() } catch { scheduleNextReadAck(); return } guard let ack = NostrEmbeddedBitChat.encodeAckForNostr(type: .readReceipt, messageID: item.receipt.originalMessageID, recipientPeerID: item.peerID, senderPeerID: senderPeerID) else { - SecureLogger.log("NostrTransport: failed to embed READ ack", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to embed READ ack", category: .session, level: .error) scheduleNextReadAck(); return } guard let event = try? NostrProtocol.createPrivateMessage(content: ack, recipientPubkey: recipientHex, senderIdentity: senderIdentity) else { - SecureLogger.log("NostrTransport: failed to build Nostr event for READ ack", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to build Nostr event for READ ack", category: .session, level: .error) scheduleNextReadAck(); return } SecureLogger.log("NostrTransport: sending READ ack giftWrap id=\(event.id.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) NostrRelayManager.shared.sendEvent(event) scheduleNextReadAck() } @@ -137,7 +137,7 @@ final class NostrTransport: Transport { guard let senderIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() else { return } let content = isFavorite ? "[FAVORITED]:\(senderIdentity.npub)" : "[UNFAVORITED]:\(senderIdentity.npub)" SecureLogger.log("NostrTransport: preparing FAVORITE(\(isFavorite)) to \(recipientNpub.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Convert recipient npub -> hex let recipientHex: String do { @@ -146,15 +146,15 @@ final class NostrTransport: Transport { recipientHex = data.hexEncodedString() } catch { return } guard let embedded = NostrEmbeddedBitChat.encodePMForNostr(content: content, messageID: UUID().uuidString, recipientPeerID: peerID, senderPeerID: senderPeerID) else { - SecureLogger.log("NostrTransport: failed to embed favorite notification", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to embed favorite notification", category: .session, level: .error) return } guard let event = try? NostrProtocol.createPrivateMessage(content: embedded, recipientPubkey: recipientHex, senderIdentity: senderIdentity) else { - SecureLogger.log("NostrTransport: failed to build Nostr event for favorite notification", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to build Nostr event for favorite notification", category: .session, level: .error) return } SecureLogger.log("NostrTransport: sending favorite giftWrap id=\(event.id.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) NostrRelayManager.shared.sendEvent(event) } } @@ -181,7 +181,7 @@ final class NostrTransport: Transport { guard let recipientNpub = resolveRecipientNpub(for: peerID) else { return } guard let senderIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() else { return } SecureLogger.log("NostrTransport: preparing DELIVERED ack for id=\(messageID.prefix(8))โ€ฆ to \(recipientNpub.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) let recipientHex: String do { let (hrp, data) = try Bech32.decode(recipientNpub) @@ -189,15 +189,15 @@ final class NostrTransport: Transport { recipientHex = data.hexEncodedString() } catch { return } guard let ack = NostrEmbeddedBitChat.encodeAckForNostr(type: .delivered, messageID: messageID, recipientPeerID: peerID, senderPeerID: senderPeerID) else { - SecureLogger.log("NostrTransport: failed to embed DELIVERED ack", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to embed DELIVERED ack", category: .session, level: .error) return } guard let event = try? NostrProtocol.createPrivateMessage(content: ack, recipientPubkey: recipientHex, senderIdentity: senderIdentity) else { - SecureLogger.log("NostrTransport: failed to build Nostr event for DELIVERED ack", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to build Nostr event for DELIVERED ack", category: .session, level: .error) return } SecureLogger.log("NostrTransport: sending DELIVERED ack giftWrap id=\(event.id.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) NostrRelayManager.shared.sendEvent(event) } } @@ -206,7 +206,7 @@ final class NostrTransport: Transport { func sendDeliveryAckGeohash(for messageID: String, toRecipientHex recipientHex: String, from identity: NostrIdentity) { Task { @MainActor in SecureLogger.log("GeoDM: send DELIVERED -> recip=\(recipientHex.prefix(8))โ€ฆ mid=\(messageID.prefix(8))โ€ฆ from=\(identity.publicKeyHex.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) guard let embedded = NostrEmbeddedBitChat.encodeAckForNostrNoRecipient(type: .delivered, messageID: messageID, senderPeerID: senderPeerID) else { return } guard let event = try? NostrProtocol.createPrivateMessage(content: embedded, recipientPubkey: recipientHex, senderIdentity: identity) else { return } NostrRelayManager.registerPendingGiftWrap(id: event.id) @@ -217,7 +217,7 @@ final class NostrTransport: Transport { func sendReadReceiptGeohash(_ messageID: String, toRecipientHex recipientHex: String, from identity: NostrIdentity) { Task { @MainActor in SecureLogger.log("GeoDM: send READ -> recip=\(recipientHex.prefix(8))โ€ฆ mid=\(messageID.prefix(8))โ€ฆ from=\(identity.publicKeyHex.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) guard let embedded = NostrEmbeddedBitChat.encodeAckForNostrNoRecipient(type: .readReceipt, messageID: messageID, senderPeerID: senderPeerID) else { return } guard let event = try? NostrProtocol.createPrivateMessage(content: embedded, recipientPubkey: recipientHex, senderIdentity: identity) else { return } NostrRelayManager.registerPendingGiftWrap(id: event.id) @@ -230,18 +230,18 @@ final class NostrTransport: Transport { Task { @MainActor in guard !recipientHex.isEmpty else { return } SecureLogger.log("GeoDM: send PM -> recip=\(recipientHex.prefix(8))โ€ฆ mid=\(messageID.prefix(8))โ€ฆ from=\(identity.publicKeyHex.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Build embedded BitChat packet without recipient peer ID guard let embedded = NostrEmbeddedBitChat.encodePMForNostrNoRecipient(content: content, messageID: messageID, senderPeerID: senderPeerID) else { - SecureLogger.log("NostrTransport: failed to embed geohash PM packet", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to embed geohash PM packet", category: .session, level: .error) return } guard let event = try? NostrProtocol.createPrivateMessage(content: embedded, recipientPubkey: recipientHex, senderIdentity: identity) else { - SecureLogger.log("NostrTransport: failed to build Nostr event for geohash PM", category: SecureLogger.session, level: .error) + SecureLogger.log("NostrTransport: failed to build Nostr event for geohash PM", category: .session, level: .error) return } SecureLogger.log("NostrTransport: sending geohash PM giftWrap id=\(event.id.prefix(16))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) NostrRelayManager.registerPendingGiftWrap(id: event.id) NostrRelayManager.shared.sendEvent(event) } diff --git a/bitchat/Services/PrivateChatManager.swift b/bitchat/Services/PrivateChatManager.swift index 41e600fb..99363b31 100644 --- a/bitchat/Services/PrivateChatManager.swift +++ b/bitchat/Services/PrivateChatManager.swift @@ -229,7 +229,7 @@ final class PrivateChatManager: ObservableObject { // Route via MessageRouter to avoid handshakeRequired spam when session isn't established if let router = messageRouter { SecureLogger.log("PrivateChatManager: sending READ ack for \(message.id.prefix(8))โ€ฆ to \(senderPeerID.prefix(8))โ€ฆ via router", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) Task { @MainActor in router.sendReadReceipt(receipt, to: senderPeerID) } diff --git a/bitchat/Services/Tor/TorManager.swift b/bitchat/Services/Tor/TorManager.swift index 8189514c..9843e9b4 100644 --- a/bitchat/Services/Tor/TorManager.swift +++ b/bitchat/Services/Tor/TorManager.swift @@ -189,7 +189,7 @@ final class TorManager: ObservableObject { var started = false // If already running (per C glue), treat as started if tor_host_is_running() != 0 { - SecureLogger.log("TorManager: embed reports already running", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: embed reports already running", category: .session, level: .info) return true } dir.withCString { dptr in @@ -198,9 +198,9 @@ final class TorManager: ObservableObject { let rc = tor_host_start(dptr, sptr, cptr, 1) started = (rc == 0) if rc != 0 { - SecureLogger.log("TorManager: tor_host_start failed rc=\(rc)", category: SecureLogger.session, level: .error) + SecureLogger.log("TorManager: tor_host_start failed rc=\(rc)", category: .session, level: .error) } else { - SecureLogger.log("TorManager: tor_host_start OK (\(socks), control \(control))", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: tor_host_start OK (\(socks), control \(control))", category: .session, level: .info) } } } @@ -215,10 +215,10 @@ final class TorManager: ObservableObject { await MainActor.run { self.socksReady = ready if ready { - SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort) [embed]", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort) [embed]", category: .session, level: .info) } else { self.lastError = NSError(domain: "TorManager", code: -14, userInfo: [NSLocalizedDescriptionKey: "Tor SOCKS not reachable after embed start"]) - SecureLogger.log("TorManager: SOCKS not reachable (timeout) [embed]", category: SecureLogger.session, level: .error) + SecureLogger.log("TorManager: SOCKS not reachable (timeout) [embed]", category: .session, level: .error) } } } @@ -282,13 +282,13 @@ final class TorManager: ObservableObject { /// Returns true if the attempt started and port probing was scheduled. private func startTorViaDlopen() -> Bool { guard let fwURL = frameworkBinaryURL() else { - SecureLogger.log("TorManager: no embedded tor framework found", category: SecureLogger.session, level: .warning) + SecureLogger.log("TorManager: no embedded tor framework found", category: .session, level: .warning) return false } // Load the library let mode = RTLD_NOW | RTLD_LOCAL - SecureLogger.log("TorManager: dlopen(\(fwURL.lastPathComponent))โ€ฆ", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: dlopen(\(fwURL.lastPathComponent))โ€ฆ", category: .session, level: .info) guard let handle = dlopen(fwURL.path, mode) else { let err = String(cString: dlerror()) self.lastError = NSError(domain: "TorManager", code: -10, userInfo: [NSLocalizedDescriptionKey: "dlopen failed: \(err)"]) @@ -314,7 +314,7 @@ final class TorManager: ObservableObject { argv.append(contentsOf: ["-f", torrc]) } // Run Tor on a background thread to avoid blocking the main actor - SecureLogger.log("TorManager: launching tor_main with torrc", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: launching tor_main with torrc", category: .session, level: .info) let argc = Int32(argv.count) DispatchQueue.global(qos: .utility).async { // Build stable C argv in this thread @@ -339,9 +339,9 @@ final class TorManager: ObservableObject { self.socksReady = ready if !ready { self.lastError = NSError(domain: "TorManager", code: -12, userInfo: [NSLocalizedDescriptionKey: "Tor SOCKS not reachable after dlopen start"]) - SecureLogger.log("TorManager: SOCKS not reachable (timeout)", category: SecureLogger.session, level: .error) + SecureLogger.log("TorManager: SOCKS not reachable (timeout)", category: .session, level: .error) } else { - SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort)", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort)", category: .session, level: .info) } // isStarting will be cleared when bootstrap reaches 100% } @@ -385,7 +385,7 @@ final class TorManager: ObservableObject { var argv: [String] = ["tor"] if let torrc = torrcURL()?.path { argv.append(contentsOf: ["-f", torrc]) } - SecureLogger.log("TorManager: starting tor_main (static)", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: starting tor_main (static)", category: .session, level: .info) let argc = Int32(argv.count) DispatchQueue.global(qos: .utility).async { // Build stable C argv in this thread @@ -409,10 +409,10 @@ final class TorManager: ObservableObject { await MainActor.run { self.socksReady = ready if ready { - SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort)", category: SecureLogger.session, level: .info) + SecureLogger.log("TorManager: SOCKS ready at \(self.socksHost):\(self.socksPort)", category: .session, level: .info) } else { self.lastError = NSError(domain: "TorManager", code: -13, userInfo: [NSLocalizedDescriptionKey: "Tor SOCKS not reachable after static start"]) - SecureLogger.log("TorManager: SOCKS not reachable (timeout)", category: SecureLogger.session, level: .error) + SecureLogger.log("TorManager: SOCKS not reachable (timeout)", category: .session, level: .error) } // isStarting will be cleared when bootstrap reaches 100% } diff --git a/bitchat/Services/UnifiedPeerService.swift b/bitchat/Services/UnifiedPeerService.swift index 87813d90..80cbafc0 100644 --- a/bitchat/Services/UnifiedPeerService.swift +++ b/bitchat/Services/UnifiedPeerService.swift @@ -205,7 +205,7 @@ final class UnifiedPeerService: ObservableObject, TransportPeerEventsDelegate { let noiseKey = peerInfo.noisePublicKey { SecureLogger.log( "๐Ÿ”„ Found favorite for '\(peerInfo.nickname)' by nickname, updating noise key", - category: SecureLogger.session, + category: .session, level: .debug ) @@ -283,7 +283,7 @@ final class UnifiedPeerService: ObservableObject, TransportPeerEventsDelegate { func toggleFavorite(_ peerID: String) { guard let peer = getPeer(by: peerID) else { SecureLogger.log("โš ๏ธ Cannot toggle favorite - peer not found: \(peerID)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return } @@ -294,14 +294,14 @@ final class UnifiedPeerService: ObservableObject, TransportPeerEventsDelegate { // Debug logging to understand the issue SecureLogger.log("๐Ÿ” Toggle favorite - peer.nickname: '\(peer.nickname)', peer.displayName: '\(peer.displayName)', peerID: \(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) if actualNickname.isEmpty { // Try to get from mesh service's current peer list if let meshPeerNickname = meshService.peerNickname(peerID: peerID) { actualNickname = meshPeerNickname SecureLogger.log("๐Ÿ” Got nickname from mesh service: '\(actualNickname)'", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -329,7 +329,7 @@ final class UnifiedPeerService: ObservableObject, TransportPeerEventsDelegate { // Log the final nickname being saved SecureLogger.log("โญ๏ธ Toggled favorite for '\(finalNickname)' (peerID: \(peerID), was: \(wasFavorite), now: \(!wasFavorite))", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Send favorite notification to the peer via router (mesh or Nostr) if let router = messageRouter { diff --git a/bitchat/Utils/OSLog+Categories.swift b/bitchat/Utils/OSLog+Categories.swift new file mode 100644 index 00000000..7172bf82 --- /dev/null +++ b/bitchat/Utils/OSLog+Categories.swift @@ -0,0 +1,20 @@ +// +// OSLog+Categories.swift +// bitchat +// +// This is free and unencumbered software released into the public domain. +// For more information, see +// + +import os.log + +extension OSLog { + private static let subsystem = "chat.bitchat" + + static let noise = OSLog(subsystem: subsystem, category: "noise") + static let encryption = OSLog(subsystem: subsystem, category: "encryption") + static let keychain = OSLog(subsystem: subsystem, category: "keychain") + static let session = OSLog(subsystem: subsystem, category: "session") + static let security = OSLog(subsystem: subsystem, category: "security") + static let handshake = OSLog(subsystem: subsystem, category: "handshake") +} diff --git a/bitchat/Utils/SecureLogger.swift b/bitchat/Utils/SecureLogger.swift index 41f0f478..30f09319 100644 --- a/bitchat/Utils/SecureLogger.swift +++ b/bitchat/Utils/SecureLogger.swift @@ -13,17 +13,6 @@ import os.log /// Provides safe logging that filters sensitive data and security events final class SecureLogger { - // MARK: - Log Categories - - private static let subsystem = "chat.bitchat" - - static let noise = OSLog(subsystem: subsystem, category: "noise") - static let encryption = OSLog(subsystem: subsystem, category: "encryption") - static let keychain = OSLog(subsystem: subsystem, category: "keychain") - static let session = OSLog(subsystem: subsystem, category: "session") - static let security = OSLog(subsystem: subsystem, category: "security") - static let handshake = OSLog(subsystem: subsystem, category: "handshake") - // MARK: - Timestamp Formatter private static let timestampFormatter: DateFormatter = { @@ -132,15 +121,15 @@ final class SecureLogger { let message = "\(location) \(event.message)" #if DEBUG - os_log("%{public}@", log: security, type: level.osLogType, message) + os_log("%{public}@", log: .security, type: level.osLogType, message) #else // In release, use private logging to prevent sensitive data exposure - os_log("%{private}@", log: security, type: level.osLogType, message) + os_log("%{private}@", log: .security, type: level.osLogType, message) #endif } /// Log general messages with automatic sensitive data filtering - static func log(_ message: @autoclosure () -> String, category: OSLog = noise, level: LogLevel = .debug, + static func log(_ message: @autoclosure () -> String, category: OSLog = .noise, level: LogLevel = .debug, file: String = #file, line: Int = #line, function: String = #function) { guard shouldLog(level) else { return } let location = formatLocation(file: file, line: line, function: function) @@ -157,7 +146,7 @@ final class SecureLogger { } /// Log errors with context - static func logError(_ error: Error, context: @autoclosure () -> String, category: OSLog = noise, + static func logError(_ error: Error, context: @autoclosure () -> String, category: OSLog = .noise, file: String = #file, line: Int = #line, function: String = #function) { let location = formatLocation(file: file, line: line, function: function) let sanitized = sanitize(context()) @@ -240,10 +229,10 @@ extension SecureLogger { static func logHandshake(_ phase: String, peerID: String, success: Bool = true, file: String = #file, line: Int = #line, function: String = #function) { if success { - log("Handshake \(phase) with peer: \(peerID)", category: session, level: .info, + log("Handshake \(phase) with peer: \(peerID)", category: .session, level: .info, file: file, line: line, function: function) } else { - log("Handshake \(phase) failed with peer: \(peerID)", category: session, level: .warning, + log("Handshake \(phase) failed with peer: \(peerID)", category: .session, level: .warning, file: file, line: line, function: function) } } @@ -253,7 +242,7 @@ extension SecureLogger { file: String = #file, line: Int = #line, function: String = #function) { let level: LogLevel = success ? .debug : .error log("Encryption operation '\(operation)' \(success ? "succeeded" : "failed")", - category: encryption, level: level, file: file, line: line, function: function) + category: .encryption, level: level, file: file, line: line, function: function) } /// Log key management operations @@ -261,7 +250,7 @@ extension SecureLogger { file: String = #file, line: Int = #line, function: String = #function) { let level: LogLevel = success ? .debug : .error log("Key operation '\(operation)' for \(keyType) \(success ? "succeeded" : "failed")", - category: keychain, level: level, file: file, line: line, function: function) + category: .keychain, level: level, file: file, line: line, function: function) } } diff --git a/bitchat/ViewModels/ChatViewModel.swift b/bitchat/ViewModels/ChatViewModel.swift index e774dd9a..178b0a6c 100644 --- a/bitchat/ViewModels/ChatViewModel.swift +++ b/bitchat/ViewModels/ChatViewModel.swift @@ -463,7 +463,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { UserDefaults.standard.set(data, forKey: "sentReadReceipts") } else { SecureLogger.log("โŒ Failed to encode read receipts for persistence", - category: SecureLogger.session, level: .error) + category: .session, level: .error) } } } @@ -562,11 +562,11 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Wait for Tor readiness before Nostr init let ready = await TorManager.shared.awaitReady(timeout: 60) guard ready else { - SecureLogger.log("Nostr init skipped: Tor not ready", category: SecureLogger.session, level: .error) + SecureLogger.log("Nostr init skipped: Tor not ready", category: .session, level: .error) return } nostrRelayManager = NostrRelayManager.shared - SecureLogger.log("Initializing Nostr relay connections", category: SecureLogger.session, level: .debug) + SecureLogger.log("Initializing Nostr relay connections", category: .session, level: .debug) // Connect is managed centrally on scene activation; avoid duplicate connects here // Attempt to flush any queued outbox after Nostr comes online @@ -597,7 +597,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { uniquePeers[peer.id] = peer } else { SecureLogger.log("โš ๏ธ Duplicate peer ID detected: \(peer.id) (\(peer.displayName))", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } self.peerIndex = uniquePeers @@ -1036,10 +1036,10 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.privateChats[convKey]?[idx].deliveryStatus = .delivered(to: self.displayNameForNostrPubkey(senderPubkey), at: Date()) self.objectWillChange.send() SecureLogger.log("GeoDM: recv DELIVERED for mid=\(messageID.prefix(8))โ€ฆ from=\(senderPubkey.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else { SecureLogger.log("GeoDM: delivered ack for unknown mid=\(messageID.prefix(8))โ€ฆ conv=\(convKey)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } case .readReceipt: @@ -1048,10 +1048,10 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.privateChats[convKey]?[idx].deliveryStatus = .read(by: self.displayNameForNostrPubkey(senderPubkey), at: Date()) self.objectWillChange.send() SecureLogger.log("GeoDM: recv READ for mid=\(messageID.prefix(8))โ€ฆ from=\(senderPubkey.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else { SecureLogger.log("GeoDM: read ack for unknown mid=\(messageID.prefix(8))โ€ฆ conv=\(convKey)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } case .verifyChallenge, .verifyResponse: @@ -1464,14 +1464,14 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { count: TransportConfig.nostrGeoRelayCount ) if targetRelays.isEmpty { - SecureLogger.log("Geo: no geohash relays available for \(ch.geohash); not sending", category: SecureLogger.session, level: .warning) + SecureLogger.log("Geo: no geohash relays available for \(ch.geohash); not sending", category: .session, level: .warning) } else { NostrRelayManager.shared.sendEvent(event, to: targetRelays) } // Track ourselves as active participant self.recordGeoParticipant(pubkeyHex: identity.publicKeyHex) SecureLogger.log("GeoTeleport: sent geo message pub=\(identity.publicKeyHex.prefix(8))โ€ฆ teleported=\(LocationChannelManager.shared.teleported)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // If we tagged this as teleported, also mark our pubkey in teleportedGeo for UI // Only when not in our regional set (and regional list is known) let hasRegional = !LocationChannelManager.shared.availableChannels.isEmpty @@ -1480,10 +1480,10 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { let key = identity.publicKeyHex.lowercased() self.teleportedGeo = self.teleportedGeo.union([key]) SecureLogger.log("GeoTeleport: mark self teleported key=\(key.prefix(8))โ€ฆ total=\(self.teleportedGeo.count)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } catch { - SecureLogger.log("โŒ Failed to send geohash message: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to send geohash message: \(error)", category: .session, level: .error) self.addSystemMessage("failed to send to location channel") } } @@ -1510,7 +1510,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Debug: log if any empty messages are present let emptyMesh = messages.filter { $0.content.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty }.count if emptyMesh > 0 { - SecureLogger.log("RenderGuard: mesh timeline contains \(emptyMesh) empty messages", category: SecureLogger.session, level: .debug) + SecureLogger.log("RenderGuard: mesh timeline contains \(emptyMesh) empty messages", category: .session, level: .debug) } stopGeoParticipantsTimer() geohashPeople = [] @@ -1537,7 +1537,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Debug: log if any empty messages are present post-sanitize let emptyGeo = messages.filter { $0.content.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty }.count if emptyGeo > 0 { - SecureLogger.log("RenderGuard: geohash \(ch.geohash) timeline has \(emptyGeo) empty messages after sanitize", category: SecureLogger.session, level: .debug) + SecureLogger.log("RenderGuard: geohash \(ch.geohash) timeline has \(emptyGeo) empty messages after sanitize", category: .session, level: .debug) } } // If switching to a location channel, flush any pending geohash-only system messages @@ -1570,7 +1570,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if LocationChannelManager.shared.teleported && hasRegional && !inRegional { teleportedGeo = teleportedGeo.union([key]) SecureLogger.log("GeoTeleport: channel switch mark self teleported key=\(key.prefix(8))โ€ฆ total=\(teleportedGeo.count)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else { teleportedGeo.remove(key) } @@ -1594,7 +1594,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Log incoming tags for diagnostics let tagSummary = event.tags.map { "[" + $0.joined(separator: ",") + "]" }.joined(separator: ",") SecureLogger.log("GeoTeleport: recv pub=\(event.pubkey.prefix(8))โ€ฆ tags=\(tagSummary)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Track teleport tag for participants โ€“ only our format ["t", "teleport"] let hasTeleportTag: Bool = event.tags.contains(where: { tag in tag.count >= 2 && tag[0].lowercased() == "t" && tag[1].lowercased() == "teleport" @@ -1612,7 +1612,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { Task { @MainActor in self.teleportedGeo = self.teleportedGeo.union([key]) SecureLogger.log("GeoTeleport: mark peer teleported key=\(key.prefix(8))โ€ฆ total=\(self.teleportedGeo.count)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } } @@ -1677,7 +1677,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Log GeoDM subscribe only when Tor is ready to avoid early noise if TorManager.shared.isReady { SecureLogger.log("GeoDM: subscribing DMs pub=\(id.publicKeyHex.prefix(8))โ€ฆ sub=\(dmSub)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } let dmFilter = NostrFilter.giftWrapsFor(pubkey: id.publicKeyHex, since: Date().addingTimeInterval(-TransportConfig.nostrDMSubscribeLookbackSeconds)) NostrRelayManager.shared.subscribe(filter: dmFilter, id: dmSub) { [weak self] giftWrap in @@ -1688,11 +1688,11 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Decrypt with per-geohash identity guard let (content, senderPubkey, rumorTs) = try? NostrProtocol.decryptPrivateMessage(giftWrap: giftWrap, recipientIdentity: id) else { SecureLogger.log("GeoDM: failed decrypt giftWrap id=\(giftWrap.id.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return } SecureLogger.log("GeoDM: decrypted gift-wrap id=\(giftWrap.id.prefix(16))... from=\(senderPubkey.prefix(8))...", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) guard content.hasPrefix("bitchat1:") else { return } guard let packetData = Self.base64URLDecode(String(content.dropFirst("bitchat1:".count))), let packet = BitchatPacket.from(packetData) else { return } @@ -1706,7 +1706,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { guard let pm = PrivateMessagePacket.decode(from: noisePayload.data) else { return } let messageId = pm.messageID SecureLogger.log("GeoDM: recv PM <- sender=\(senderPubkey.prefix(8))โ€ฆ mid=\(messageId.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // Send delivery ACK immediately (even if duplicate), once per messageID if !self.sentGeoDeliveryAcks.contains(messageId) { let nostrTransport = NostrTransport() @@ -1770,10 +1770,10 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.privateChats[convKey]?[idx].deliveryStatus = .delivered(to: self.displayNameForNostrPubkey(senderPubkey), at: Date()) self.objectWillChange.send() SecureLogger.log("GeoDM: recv DELIVERED for mid=\(messageID.prefix(8))โ€ฆ from=\(senderPubkey.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else { SecureLogger.log("GeoDM: delivered ack for unknown mid=\(messageID.prefix(8))โ€ฆ conv=\(convKey)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } case .readReceipt: @@ -1782,10 +1782,10 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.privateChats[convKey]?[idx].deliveryStatus = .read(by: self.displayNameForNostrPubkey(senderPubkey), at: Date()) self.objectWillChange.send() SecureLogger.log("GeoDM: recv READ for mid=\(messageID.prefix(8))โ€ฆ from=\(senderPubkey.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else { SecureLogger.log("GeoDM: read ack for unknown mid=\(messageID.prefix(8))โ€ฆ conv=\(convKey)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } default: @@ -2153,7 +2153,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { return } SecureLogger.log("GeoDM: local send mid=\(messageID.prefix(8))โ€ฆ to=\(recipientHex.prefix(8))โ€ฆ conv=\(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) let nostrTransport = NostrTransport() nostrTransport.senderPeerID = meshService.myPeerID nostrTransport.sendPrivateMessageGeohash(content: content, toRecipientHex: recipientHex, from: id, messageID: messageID) @@ -2474,7 +2474,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if hadUnreadTemp { unreadPrivateMessages.insert(peerID) SecureLogger.log("๐Ÿ“ฌ Transferred unread status from temp peer IDs to \(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } if consolidatedCount > 0 { @@ -2482,7 +2482,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { privateChats[peerID]?.sort { $0.timestamp < $1.timestamp } SecureLogger.log("๐Ÿ“ฅ Consolidated \(consolidatedCount) Nostr messages from temporary peer IDs to \(peerNickname)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } @@ -2497,7 +2497,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } } else { SecureLogger.log("GeoDM: skipping mesh handshake for virtual peerID=\(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } // Delegate to private chat manager but add already-acked messages first @@ -2578,7 +2578,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { guard let receipt = notification.userInfo?["receipt"] as? ReadReceipt else { return } SecureLogger.log("๐Ÿ“– Handling read receipt for message \(receipt.originalMessageID) from Nostr", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // Process the read receipt through the same flow as Bluetooth read receipts didReceiveReadReceipt(receipt) @@ -2604,7 +2604,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // If we have a private chat open with the old peer ID, update it to the new one if selectedPrivateChatPeer == oldPeerID { SecureLogger.log("๐Ÿ“ฑ Updating private chat peer ID due to key change: \(oldPeerID) -> \(newPeerID)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // Transfer private chat messages to new peer ID if let messages = privateChats[oldPeerID] { @@ -2636,7 +2636,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Even if the chat isn't open, migrate any existing private chat data if let messages = privateChats[oldPeerID] { SecureLogger.log("๐Ÿ“ฑ Migrating private chat messages from \(oldPeerID) to \(newPeerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) var chats = privateChats chats[newPeerID] = messages chats.removeValue(forKey: oldPeerID) @@ -2742,7 +2742,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { messageRouter.sendPrivate(screenshotMessage, to: peerID, recipientNickname: peerNickname, messageID: UUID().uuidString) default: // Don't send screenshot notification if no session exists - SecureLogger.log("Skipping screenshot notification to \(peerID) - no established session", category: SecureLogger.security, level: .debug) + SecureLogger.log("Skipping screenshot notification to \(peerID) - no established session", category: .security, level: .debug) } } @@ -2783,14 +2783,14 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { ) let targetRelays = GeoRelayDirectory.shared.closestRelays(toGeohash: ch.geohash, count: 5) if targetRelays.isEmpty { - SecureLogger.log("Geo: no geohash relays available for \(ch.geohash); not sending", category: SecureLogger.session, level: .warning) + SecureLogger.log("Geo: no geohash relays available for \(ch.geohash); not sending", category: .session, level: .warning) } else { NostrRelayManager.shared.sendEvent(event, to: targetRelays) } // Track ourselves as active participant self.recordGeoParticipant(pubkeyHex: identity.publicKeyHex) } catch { - SecureLogger.log("โŒ Failed to send geohash screenshot message: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to send geohash screenshot message: \(error)", category: .session, level: .error) self.addSystemMessage("failed to send to location channel") } } @@ -2849,7 +2849,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { for (currentPeerID, currentNickname) in meshService.getPeerNicknames() { if currentNickname == peerNickname { SecureLogger.log("๐Ÿ“– Resolved updated peer ID for read receipt: \(peerID) -> \(currentPeerID)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) actualPeerID = currentPeerID break } @@ -2878,7 +2878,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { for message in messages where message.senderPeerID == peerID && !message.isRelay { if !sentReadReceipts.contains(message.id) { SecureLogger.log("GeoDM: sending READ for mid=\(message.id.prefix(8))โ€ฆ to=\(recipientHex.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) let nostrTransport = NostrTransport() nostrTransport.senderPeerID = meshService.myPeerID nostrTransport.sendReadReceiptGeohash(message.id, toRecipientHex: recipientHex, from: id) @@ -4322,14 +4322,14 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { verifiedFingerprints = SecureIdentityStateManager.shared.getVerifiedFingerprints() // Log snapshot for debugging persistence let sample = Array(verifiedFingerprints.prefix(TransportConfig.uiFingerprintSampleCount)).map { $0.prefix(8) }.joined(separator: ", ") - SecureLogger.log("๐Ÿ” Verified loaded: \(verifiedFingerprints.count) [\(sample)]", category: SecureLogger.security, level: .info) + SecureLogger.log("๐Ÿ” Verified loaded: \(verifiedFingerprints.count) [\(sample)]", category: .security, level: .info) // Also log any offline favorites and whether we consider them verified let offlineFavorites = unifiedPeerService.favorites.filter { !$0.isConnected } for fav in offlineFavorites { let fp = unifiedPeerService.getFingerprint(for: fav.id) let isVer = fp.flatMap { verifiedFingerprints.contains($0) } ?? false let fpShort = fp?.prefix(8) ?? "nil" - SecureLogger.log("โญ๏ธ Favorite offline: \(fav.nickname) fp=\(fpShort) verified=\(isVer)", category: SecureLogger.security, level: .info) + SecureLogger.log("โญ๏ธ Favorite offline: \(fav.nickname) fp=\(fpShort) verified=\(isVer)", category: .security, level: .info) } // Invalidate cached encryption statuses so offline favorites can show verified badges immediately invalidateEncryptionCache() @@ -4345,7 +4345,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { DispatchQueue.main.async { guard let self = self else { return } - SecureLogger.log("๐Ÿ” Authenticated: \(peerID)", category: SecureLogger.security, level: .debug) + SecureLogger.log("๐Ÿ” Authenticated: \(peerID)", category: .security, level: .debug) // Update encryption status if self.verifiedFingerprints.contains(fingerprint) { @@ -4365,7 +4365,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { let stable = keyData.hexEncodedString() self.shortIDToNoiseKey[peerID] = stable SecureLogger.log("๐Ÿ—บ๏ธ Mapped short peerID to Noise key for header continuity: \(peerID) -> \(stable.prefix(8))โ€ฆ", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } // If a QR verification is pending but not sent yet, send it now that session is authenticated @@ -4373,7 +4373,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.meshService.sendVerifyChallenge(to: peerID, noiseKeyHex: pending.noiseKeyHex, nonceA: pending.nonceA) pending.sent = true self.pendingQRVerifications[peerID] = pending - SecureLogger.log("๐Ÿ“ค Sent deferred verify challenge to \(peerID) after handshake", category: SecureLogger.security, level: .debug) + SecureLogger.log("๐Ÿ“ค Sent deferred verify challenge to \(peerID) after handshake", category: .security, level: .debug) } // Schedule UI update @@ -4519,7 +4519,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { pendingQRVerifications.removeValue(forKey: peerID) if let fp = getFingerprint(for: peerID) { let short = fp.prefix(8) - SecureLogger.log("๐Ÿ” Marking verified fingerprint: \(short)", category: SecureLogger.security, level: .info) + SecureLogger.log("๐Ÿ” Marking verified fingerprint: \(short)", category: .security, level: .info) SecureIdentityStateManager.shared.setVerified(fingerprint: fp, verified: true) SecureIdentityStateManager.shared.forceSave() verifiedFingerprints.insert(fp) @@ -4605,7 +4605,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // MARK: - Peer Connection Events func didConnectToPeer(_ peerID: String) { - SecureLogger.log("๐Ÿค Peer connected: \(peerID)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿค Peer connected: \(peerID)", category: .session, level: .debug) // Handle all main actor work async Task { @MainActor in @@ -4623,7 +4623,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { try? await Task.sleep(nanoseconds: TransportConfig.uiAsyncMediumSleepNs) // 0.5 seconds meshService.sendFavoriteNotification(to: peerID, isFavorite: true) SecureLogger.log("๐Ÿ“ค Resent favorite notification to reconnected peer \(peerID)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } // Force UI refresh @@ -4643,7 +4643,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } func didDisconnectFromPeer(_ peerID: String) { - SecureLogger.log("๐Ÿ‘‹ Peer disconnected: \(peerID)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ‘‹ Peer disconnected: \(peerID)", category: .session, level: .debug) // Remove ephemeral session from identity manager SecureIdentityStateManager.shared.removeEphemeralSession(peerID: peerID) @@ -4739,7 +4739,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.recentlySeenPeers = currentPeerSet NotificationService.shared.sendNetworkAvailableNotification(peerCount: meshPeers.count) SecureLogger.log("๐Ÿ‘ฅ Sent bitchatters nearby notification for \(meshPeers.count) mesh peers", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } else { // No peers โ€” immediately reset to allow next rising-edge to notify @@ -4749,7 +4749,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { self.networkResetTimer?.invalidate() self.networkResetTimer = nil } - SecureLogger.log("โณ Mesh empty โ€” reset network notification state", category: SecureLogger.session, level: .debug) + SecureLogger.log("โณ Mesh empty โ€” reset network notification state", category: .session, level: .debug) } // Register ephemeral sessions for all connected peers @@ -4812,7 +4812,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if !idsToRemove.isEmpty { SecureLogger.log("๐Ÿงน Cleaned up \(idsToRemove.count) stale unread peer IDs", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -4843,7 +4843,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { let removedCount = oldCount - sentReadReceipts.count if removedCount > 0 { SecureLogger.log("๐Ÿงน Cleaned up \(removedCount) old read receipts", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) } } @@ -4891,7 +4891,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { nostrPublicKey = data.hexEncodedString() } } catch { - SecureLogger.log("Failed to decode Nostr npub: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to decode Nostr npub: \(error)", category: .session, level: .error) } } @@ -5032,7 +5032,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { NostrRelayManager.shared.sendEvent(event, to: targetRelays) } } catch { - SecureLogger.log("โŒ Failed to send geohash raw message: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("โŒ Failed to send geohash raw message: \(error)", category: .session, level: .error) } } return @@ -5048,12 +5048,12 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { @MainActor private func setupNostrMessageHandling() { guard let currentIdentity = try? NostrIdentityBridge.getCurrentNostrIdentity() else { - SecureLogger.log("โš ๏ธ No Nostr identity available for message handling", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ No Nostr identity available for message handling", category: .session, level: .warning) return } SecureLogger.log("๐Ÿ”‘ Setting up Nostr subscription for pubkey: \(currentIdentity.publicKeyHex.prefix(16))...", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Subscribe to Nostr messages let filter = NostrFilter.giftWrapsFor( @@ -5093,19 +5093,19 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Expect embedded BitChat packet content guard content.hasPrefix("bitchat1:") else { - SecureLogger.log("Ignoring non-embedded Nostr DM content", category: SecureLogger.session, level: .debug) + SecureLogger.log("Ignoring non-embedded Nostr DM content", category: .session, level: .debug) return } guard let packetData = Self.base64URLDecode(String(content.dropFirst("bitchat1:".count))), let packet = BitchatPacket.from(packetData) else { - SecureLogger.log("Failed to decode embedded BitChat packet from Nostr DM", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to decode embedded BitChat packet from Nostr DM", category: .session, level: .error) return } // Only process typed noiseEncrypted envelope for private messages/receipts guard packet.type == MessageType.noiseEncrypted.rawValue else { - SecureLogger.log("Unsupported embedded packet type: \(packet.type)", category: SecureLogger.session, level: .warning) + SecureLogger.log("Unsupported embedded packet type: \(packet.type)", category: .session, level: .warning) return } @@ -5119,7 +5119,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Parse plaintext typed payload guard let noisePayload = NoisePayload.decode(packet.payload) else { - SecureLogger.log("Failed to parse embedded NoisePayload", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to parse embedded NoisePayload", category: .session, level: .error) return } @@ -5215,14 +5215,14 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Send delivery ack via Nostr embedded if !wasReadBefore { if let key = actualSenderNoiseKey { - SecureLogger.log("Sending DELIVERED ack for \(messageId.prefix(8))โ€ฆ via router", category: SecureLogger.session, level: .debug) + SecureLogger.log("Sending DELIVERED ack for \(messageId.prefix(8))โ€ฆ via router", category: .session, level: .debug) messageRouter.sendDeliveryAck(messageId, to: key.hexEncodedString()) } else if let id = try? NostrIdentityBridge.getCurrentNostrIdentity() { // Fallback: no Noise mapping yet โ€” send directly to sender's Nostr pubkey let nt = NostrTransport() nt.senderPeerID = meshService.myPeerID nt.sendDeliveryAckGeohash(for: messageId, toRecipientHex: senderPubkey, from: id) - SecureLogger.log("Sent DELIVERED ack directly to Nostr pub=\(senderPubkey.prefix(8))โ€ฆ for mid=\(messageId.prefix(8))โ€ฆ", category: SecureLogger.session, level: .debug) + SecureLogger.log("Sent DELIVERED ack directly to Nostr pub=\(senderPubkey.prefix(8))โ€ฆ for mid=\(messageId.prefix(8))โ€ฆ", category: .session, level: .debug) } } @@ -5237,7 +5237,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if !sentReadReceipts.contains(messageId) { if let key = actualSenderNoiseKey { let receipt = ReadReceipt(originalMessageID: messageId, readerID: meshService.myPeerID, readerNickname: nickname) - SecureLogger.log("Viewing chat; sending READ ack for \(messageId.prefix(8))โ€ฆ via router", category: SecureLogger.session, level: .debug) + SecureLogger.log("Viewing chat; sending READ ack for \(messageId.prefix(8))โ€ฆ via router", category: .session, level: .debug) messageRouter.sendReadReceipt(receipt, to: key.hexEncodedString()) sentReadReceipts.insert(messageId) } else if let id = try? NostrIdentityBridge.getCurrentNostrIdentity() { @@ -5245,7 +5245,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { nt.senderPeerID = meshService.myPeerID nt.sendReadReceiptGeohash(messageId, toRecipientHex: senderPubkey, from: id) sentReadReceipts.insert(messageId) - SecureLogger.log("Viewing chat; sent READ ack directly to Nostr pub=\(senderPubkey.prefix(8))โ€ฆ for mid=\(messageId.prefix(8))โ€ฆ", category: SecureLogger.session, level: .debug) + SecureLogger.log("Viewing chat; sent READ ack directly to Nostr pub=\(senderPubkey.prefix(8))โ€ฆ for mid=\(messageId.prefix(8))โ€ฆ", category: .session, level: .debug) } } } else { @@ -5290,7 +5290,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } } catch { - SecureLogger.log("Failed to decrypt Nostr message: \(error)", category: SecureLogger.session, level: .error) + SecureLogger.log("Failed to decrypt Nostr message: \(error)", category: .session, level: .error) } } @@ -5311,7 +5311,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Parse ACK format: "ACK:TYPE:MESSAGE_ID" let parts = content.split(separator: ":", maxSplits: 2) guard parts.count >= 3 else { - SecureLogger.log("โš ๏ธ Invalid ACK format: \(content)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Invalid ACK format: \(content)", category: .session, level: .warning) return } @@ -5327,7 +5327,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { processedNostrAcks.insert(ackKey) SecureLogger.log("๐Ÿ“จ Received \(ackType) ACK for message \(messageId.prefix(16))... from \(senderPubkey.prefix(16))...", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) // Verify the sender has a valid Noise key guard findNoiseKey(for: senderPubkey) != nil else { @@ -5346,12 +5346,12 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { case "READ": privateChats[chatPeerID]?[index].deliveryStatus = .read(by: "recipient", at: Date()) default: - SecureLogger.log("โš ๏ธ Unknown ACK type: \(ackType)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Unknown ACK type: \(ackType)", category: .session, level: .warning) } messageFound = true SecureLogger.log("โœ… Updated message \(messageId.prefix(16))... status to \(ackType) in chat \(chatPeerID.prefix(16))...", - category: SecureLogger.session, level: .info) + category: .session, level: .info) // Don't break - continue to update in all chats where this message exists } } @@ -5360,7 +5360,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { objectWillChange.send() } else { SecureLogger.log("โš ๏ธ Could not find message \(messageId) to update status from ACK", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } @@ -5387,7 +5387,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if parts.count > 1 { nostrPubkey = String(parts[1]) SecureLogger.log("๐Ÿ“ Received Nostr npub in favorite notification: \(nostrPubkey ?? "none")", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } // Get the noise public key for this peer @@ -5406,7 +5406,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { guard let finalNoiseKey = noiseKey else { SecureLogger.log("โš ๏ธ Cannot get Noise key for peer \(peerID)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return } @@ -5421,7 +5421,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // If they favorited us and provided their Nostr key, ensure it's stored if isFavorite && nostrPubkey != nil { SecureLogger.log("๐Ÿ’พ Storing Nostr key association for \(senderNickname): \(nostrPubkey!.prefix(16))...", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } // Show system message @@ -5533,7 +5533,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } SecureLogger.log("๐Ÿ“ฌ Stored Nostr message from unknown sender \(finalSenderNickname) in temporary peer \(tempPeerID)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } @MainActor @@ -5546,7 +5546,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Try to convert hex to npub guard let pubkeyData = Data(hexString: nostrPubkey) else { SecureLogger.log("โš ๏ธ Invalid hex public key format: \(nostrPubkey.prefix(16))...", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return nil } @@ -5554,7 +5554,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { npubToMatch = try Bech32.encode(hrp: "npub", data: pubkeyData) } catch { SecureLogger.log("โš ๏ธ Failed to convert hex to npub: \(error)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) return nil } } @@ -5565,21 +5565,21 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Compare npub format if storedNostrKey == npubToMatch { // SecureLogger.log("โœ… Found Noise key for Nostr sender (npub match)", - // category: SecureLogger.session, level: .debug) + // category: .session, level: .debug) return noiseKey } // Also try hex comparison if stored value is hex if !storedNostrKey.hasPrefix("npub") && storedNostrKey == nostrPubkey { SecureLogger.log("โœ… Found Noise key for Nostr sender (hex match)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) return noiseKey } } } SecureLogger.log("โš ๏ธ No matching Noise key found for Nostr pubkey: \(nostrPubkey.prefix(16))... (tried npub: \(npubToMatch.prefix(16))...)", - category: SecureLogger.session, level: .debug) + category: .session, level: .debug) return nil } @@ -5607,13 +5607,13 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { // Try mesh first for connected peers if meshService.isPeerConnected(peerID) { messageRouter.sendFavoriteNotification(to: peerID, isFavorite: isFavorite) - SecureLogger.log("๐Ÿ“ค Sent favorite notification via BLE to \(peerID)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ค Sent favorite notification via BLE to \(peerID)", category: .session, level: .debug) } else if let key = noiseKey { // Send via Nostr for offline peers (using router) let recipientPeerID = key.hexEncodedString() messageRouter.sendFavoriteNotification(to: recipientPeerID, isFavorite: isFavorite) } else { - SecureLogger.log("โš ๏ธ Cannot send favorite notification - peer not connected and no Nostr pubkey", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Cannot send favorite notification - peer not connected and no Nostr pubkey", category: .session, level: .warning) } } @@ -5704,11 +5704,11 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } else { // Keep old messages in original location but don't show in UI SecureLogger.log("๐Ÿ“ฆ Partially migrating \(recentMessages.count) of \(messages.count) messages from \(oldPeerID)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } SecureLogger.log("๐Ÿ“ฆ Migrating \(recentMessages.count) recent messages from old peer ID \(oldPeerID) to \(peerID) (fingerprint match)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } else if currentFingerprint == nil || oldFingerprint == nil { // Check if this chat contains messages with this sender by nickname let isRelevantChat = recentMessages.contains { msg in @@ -5725,7 +5725,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { } SecureLogger.log("๐Ÿ“ฆ Migrating \(recentMessages.count) recent messages from old peer ID \(oldPeerID) to \(peerID) (nickname match)", - category: SecureLogger.session, level: .warning) + category: .session, level: .warning) } } } @@ -5765,7 +5765,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { if needsSelectedUpdate { selectedPrivateChatPeer = peerID SecureLogger.log("๐Ÿ“ฑ Updated selectedPrivateChatPeer from old ID to \(peerID) during migration", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } } @@ -5774,11 +5774,11 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { /// Handle incoming private message @MainActor private func handlePrivateMessage(_ message: BitchatMessage) { - SecureLogger.log("๐Ÿ“ฅ handlePrivateMessage called for message from \(message.sender)", category: SecureLogger.session, level: .debug) + SecureLogger.log("๐Ÿ“ฅ handlePrivateMessage called for message from \(message.sender)", category: .session, level: .debug) let senderPeerID = message.senderPeerID ?? getPeerIDForNickname(message.sender) guard let peerID = senderPeerID else { - SecureLogger.log("โš ๏ธ Could not get peer ID for sender \(message.sender)", category: SecureLogger.session, level: .warning) + SecureLogger.log("โš ๏ธ Could not get peer ID for sender \(message.sender)", category: .session, level: .warning) return } @@ -5819,7 +5819,7 @@ final class ChatViewModel: ObservableObject, BitchatDelegate { privateChats.removeValue(forKey: stableKeyHex) SecureLogger.log("๐Ÿ“ฅ Consolidated \(nostrMessages.count) Nostr messages from stable key to ephemeral peer \(peerID)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) } } } @@ -6087,7 +6087,7 @@ private func checkForMentions(_ message: BitchatMessage) { if isMentioned && message.sender != nickname { SecureLogger.log("๐Ÿ”” Mention from \(message.sender)", - category: SecureLogger.session, level: .info) + category: .session, level: .info) NotificationService.shared.sendMentionNotification(from: message.sender, message: message.content) } }