Error presenter: Always handle the passed error

Signed-off-by: Roopesh Chander <roop@roopc.net>

diff --git a/WireGuard/WireGuard/UI/iOS/ErrorPresenter.swift b/WireGuard/WireGuard/UI/iOS/ErrorPresenter.swift
index 1116f6145ab54f2c452b6a56e3fb2cf53f2a1b09..2ae0cf0d720ca5bcaf1777efbd5d7e041ae3ae68 100644 (file)
--- a/WireGuard/WireGuard/UI/iOS/ErrorPresenter.swift
+++ b/WireGuard/WireGuard/UI/iOS/ErrorPresenter.swift
@@ -5,7 +5,7 @@ import UIKit
 import os.log
 
 class ErrorPresenter {
-    static func errorMessage(for error: Error) -> (String, String)? {
+    static func errorMessage(for error: Error) -> (String, String) {
         switch (error) {
 
         // TunnelManagementError
@@ -43,15 +43,14 @@ class ErrorPresenter {
             return ("Nothing to export", "There are no tunnels to export")
 
         default:
-            os_log("ErrorPresenter: Error not presented: %{public}@", log: OSLog.default, type: .error, "\(error)")
-            return nil
+            return ("Error", error.localizedDescription)
         }
     }
 
     static func showErrorAlert(error: Error, from sourceVC: UIViewController?,
                                onDismissal: (() -> Void)? = nil, onPresented: (() -> Void)? = nil) {
         guard let sourceVC = sourceVC else { return }
-        guard let (title, message) = ErrorPresenter.errorMessage(for: error) else { return }
+        let (title, message) = ErrorPresenter.errorMessage(for: error)
         let okAction = UIAlertAction(title: "OK", style: .default) { (_) in
             onDismissal?()
         }