Browse Source

When we get a consensus that's been signed by more people than
we expect, don't log about it; it's not a big deal.


svn:r13249

Roger Dingledine 16 years ago
parent
commit
980fcb1ca7
2 changed files with 7 additions and 5 deletions
  1. 2 0
      ChangeLog
  2. 5 5
      src/or/networkstatus.c

+ 2 - 0
ChangeLog

@@ -24,6 +24,8 @@ Changes in version 0.2.0.18-alpha - 2008-01-??
   o Minor bugfixes:
     - When we setconf ClientOnly to 1, close any current OR and Dir
       listeners. Reported by mwenge.
+    - When we get a consensus that's been signed by more people than
+      we expect, don't log about it; it's not a big deal.
 
   o Minor features:
     - Don't answer "/tor/networkstatus-bridges" directory requests if

+ 5 - 5
src/or/networkstatus.c

@@ -435,11 +435,11 @@ networkstatus_check_consensus_signature(networkstatus_vote_t *consensus,
   if (warn >= 0) {
     SMARTLIST_FOREACH(unrecognized, networkstatus_voter_info_t *, voter,
       {
-        log(severity, LD_DIR, "Consensus includes unrecognized authority '%s' "
-            "at %s:%d (contact %s; identity %s)",
-            voter->nickname, voter->address, (int)voter->dir_port,
-            voter->contact?voter->contact:"n/a",
-            hex_str(voter->identity_digest, DIGEST_LEN));
+        log_info(LD_DIR, "Consensus includes unrecognized authority '%s' "
+                 "at %s:%d (contact %s; identity %s)",
+                 voter->nickname, voter->address, (int)voter->dir_port,
+                 voter->contact?voter->contact:"n/a",
+                 hex_str(voter->identity_digest, DIGEST_LEN));
       });
     SMARTLIST_FOREACH(need_certs_from, networkstatus_voter_info_t *, voter,
       {