tor

commit 873d6d3724a4f83e010a8e25f7191bd4ddfa4703
parent e93ad428e2507f676ce97450b919c2d849633669
Author: Nick Mathewson <nickm@torproject.org>
Date:   Wed,  1 Jul 2020 15:41:32 -0400

Clean up a redundant debug log in circuit_handle_first_hop().

We don't need to log that we're about to look for a channel for a
given extend_info_t, since we're either going to log that we're
launching one (at info), or that we're using an existing one (at
debug).

Diffstat:
M
src/core/or/circuitbuild.c
 | 
7
++-----

1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/core/or/circuitbuild.c b/src/core/or/circuitbuild.c
@@ -561,10 +561,6 @@ circuit_handle_first_hop(origin_circuit_t *circ)
   }
 
   /* now see if we're already connected to the first OR in 'route' */
-  // TODO XXXX S55 -- remove this log
-  log_debug(LD_CIRC,"Looking for firsthop for %s",
-            extend_info_describe(firsthop->extend_info));
-
   const tor_addr_port_t *orport4 =
     extend_info_get_orport(firsthop->extend_info, AF_INET);
   const tor_addr_port_t *orport6 =
@@ -603,7 +599,8 @@ circuit_handle_first_hop(origin_circuit_t *circ)
     tor_assert(!circ->base_.n_hop);
     circ->base_.n_chan = n_chan;
     circuit_chan_publish(circ, n_chan);
-    log_debug(LD_CIRC,"Conn open. Delivering first onion skin.");
+    log_debug(LD_CIRC,"Conn open for %s. Delivering first onion skin.",
+              safe_str_client(extend_info_describe(firsthop->extend_info)));
     if ((err_reason = circuit_send_next_onion_skin(circ)) < 0) {
       log_info(LD_CIRC,"circuit_send_next_onion_skin failed.");
       circ->base_.n_chan = NULL;