Merge pull request #363 from ianco/fix_retry_logs

Fix logging on http retries
openwallet-foundation · Feb 6, 2020 · 339ce1f · 339ce1f
2 parents 500b678 + 1402026
commit 339ce1f
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/aries_cloudagent/transport/outbound/manager.py b/aries_cloudagent/transport/outbound/manager.py
@@ -407,15 +407,17 @@ def finished_deliver(self, queued: QueuedOutboundMessage, completed: CompletedTa
         """Handle completion of queued message delivery."""
         if completed.exc_info:
             queued.error = completed.exc_info
-            LOGGER.exception(
-                "Outbound message could not be delivered", exc_info=queued.error,
-            )
 
             if queued.retries:
+                LOGGER.error(">>> Posting error; Re-queue failed message ...")
                 queued.retries -= 1
                 queued.state = QueuedOutboundMessage.STATE_RETRY
                 queued.retry_at = time.perf_counter() + 10
             else:
+                LOGGER.exception(
+                    "Outbound message could not be delivered", exc_info=queued.error,
+                )
+                LOGGER.error(">>> NOT Re-queued, state is DONE, failed to deliver msg.")
                 queued.state = QueuedOutboundMessage.STATE_DONE
         else:
             queued.error = None