Split error and compltion message.

tonystone · May 23, 2018 · a1c142c · a1c142c
1 parent 2d5c627
commit a1c142c
Showing 1 changed file with 3 additions and 10 deletions.
diff --git a/Sources/Connect/Internal/Queues/ActionContianer.swift b/Sources/Connect/Internal/Queues/ActionContianer.swift
@@ -75,16 +75,7 @@ internal class ActionContainer: Operation, ActionProxy {
         defer {
             self.state = .finished
 
-            logInfo(Log.tag) {
-                var message = "Proxy \(self) \(self.completionStatus)"
-
-                if self.completionStatus == .failed,  let error = self.error {
-                    message.append( " with error: \(error).")
-                } else {
-                    message.append( ", execution statistics: \(self.statistics)")
-                }
-                return message
-            }
+            logInfo(Log.tag) { "Proxy \(self) \(self.completionStatus), execution statistics: \(self.statistics)" }
 
             self.completion?(self)
         }
@@ -115,6 +106,8 @@ internal class ActionContainer: Operation, ActionProxy {
             if completionStatus != .canceled {
                 completionStatus = .failed
             }
+
+            logError(Log.tag) { "Action \(self.action) threw error: \(error)." }
         }
     }