Commit 2b8962bc authored by Nick Mathewson's avatar Nick Mathewson 🤹
Browse files

Move onion-type stats message into heartbeat

Fix for 10485. Fix on 0.2.4.17-alpha.
parent 35115496
Loading
Loading
Loading
Loading

changes/bug10485

0 → 100644
+4 −0
Original line number Diff line number Diff line
  o Minor bugfixes:
    - Turn "circuit handshake stats since last time" log messages into a
      heartbeat message. Fixes bug 10485; bugfix on 0.2.4.17-rc.
+0 −5
Original line number Diff line number Diff line
@@ -1353,11 +1353,6 @@ run_scheduled_events(time_t now)
        next_time_to_write_stats_files = next_write;
    }
    time_to_write_stats_files = next_time_to_write_stats_files;

    /* Also commandeer this opportunity to log how our circuit handshake
     * stats have been doing. */
    if (public_server_mode(options))
      rep_hist_log_circuit_handshake_stats(now);
  }

  /* 1h. Check whether we should write bridge statistics to disk.
+1 −1
Original line number Diff line number Diff line
@@ -3042,7 +3042,7 @@ rep_hist_log_circuit_handshake_stats(time_t now)
{
  (void)now;
  /* XXX024 maybe quiet this log message before 0.2.4 goes stable for real */
  log_notice(LD_HIST, "Circuit handshake stats since last time: "
  log_notice(LD_HEARTBEAT, "Circuit handshake stats since last time: "
             "%d/%d TAP, %d/%d NTor.",
             onion_handshakes_completed[ONION_HANDSHAKE_TYPE_TAP],
             onion_handshakes_requested[ONION_HANDSHAKE_TYPE_TAP],
+6 −0
Original line number Diff line number Diff line
@@ -14,6 +14,7 @@
#include "router.h"
#include "circuitlist.h"
#include "main.h"
#include "rephist.h"
#include "hibernate.h"

/** Return the total number of circuits. */
@@ -121,6 +122,11 @@ log_heartbeat(time_t now)
    log_notice(LD_HEARTBEAT, "TLS write overhead: %.f%%", overhead);
  }

  /* Also commandeer this opportunity to log how our circuit handshake
   * stats have been doing. */
  if (public_server_mode(options))
    rep_hist_log_circuit_handshake_stats(now);

  tor_free(uptime);
  tor_free(bw_sent);
  tor_free(bw_rcvd);