[tor-commits] [tor/master] Include a HOST item in BOOTSTRAP problem messages

[nickm at torproject.org]

commit 79c69d18b76e8836593e1beb3d239a5ff5e63a7b
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Feb 27 11:28:30 2015 -0500

    Include a HOST item in BOOTSTRAP problem messages
    
    Closes ticket 15006.
---
 changes/feature15006 |    4 ++++
 src/or/control.c     |   11 +++++++----
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/changes/feature15006 b/changes/feature15006
new file mode 100644
index 0000000..168a440
--- /dev/null
+++ b/changes/feature15006
@@ -0,0 +1,4 @@
+  o Minor features (controller):
+    - Messages about problems in the bootstrap process now include
+      information about the server we were trying to connect to when we
+      noticed the problem. Closes ticket 15006.
diff --git a/src/or/control.c b/src/or/control.c
index a2b9867..dc3e5fc 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -5049,6 +5049,7 @@ MOCK_IMPL(void,
   char buf[BOOTSTRAP_MSG_LEN];
   const char *recommendation = "ignore";
   int severity;
+  char host[128];
 
   /* bootstrap_percent must not be in "undefined" state here. */
   tor_assert(status >= 0);
@@ -5078,6 +5079,8 @@ MOCK_IMPL(void,
   if (we_are_hibernating())
     recommendation = "ignore";
 
+  orconn_target_get_name(host, sizeof(host), or_conn);
+
   while (status>=0 && bootstrap_status_to_string(status, &tag, &summary) < 0)
     status--; /* find a recognized status string based on current progress */
   status = bootstrap_percent; /* set status back to the actual number */
@@ -5086,19 +5089,19 @@ MOCK_IMPL(void,
 
   log_fn(severity,
          LD_CONTROL, "Problem bootstrapping. Stuck at %d%%: %s. (%s; %s; "
-         "count %d; recommendation %s)",
+         "count %d; recommendation %s; host %s)",
          status, summary, warn,
          orconn_end_reason_to_control_string(reason),
-         bootstrap_problems, recommendation);
+         bootstrap_problems, recommendation, host);
 
   connection_or_report_broken_states(severity, LD_HANDSHAKE);
 
   tor_snprintf(buf, sizeof(buf),
       "BOOTSTRAP PROGRESS=%d TAG=%s SUMMARY=\"%s\" WARNING=\"%s\" REASON=%s "
-      "COUNT=%d RECOMMENDATION=%s",
+      "COUNT=%d RECOMMENDATION=%s HOST=\"%s\"",
       bootstrap_percent, tag, summary, warn,
       orconn_end_reason_to_control_string(reason), bootstrap_problems,
-      recommendation);
+      recommendation, host);
   tor_snprintf(last_sent_bootstrap_message,
                sizeof(last_sent_bootstrap_message),
                "WARN %s", buf);