[or-cvs] r14441: tone down some of the log severities (tor/branches/tor-0_2_0-patches/src/or)

[arma at seul.org]

Author: arma
Date: 2008-04-24 01:17:58 -0400 (Thu, 24 Apr 2008)
New Revision: 14441

Modified:
   tor/branches/tor-0_2_0-patches/src/or/directory.c
   tor/branches/tor-0_2_0-patches/src/or/networkstatus.c
Log:
tone down some of the log severities


Modified: tor/branches/tor-0_2_0-patches/src/or/directory.c
===================================================================
--- tor/branches/tor-0_2_0-patches/src/or/directory.c	2008-04-24 02:43:15 UTC (rev 14440)
+++ tor/branches/tor-0_2_0-patches/src/or/directory.c	2008-04-24 05:17:58 UTC (rev 14441)
@@ -1534,7 +1534,8 @@
     if ((r=networkstatus_set_current_consensus(body, 0))<0) {
       log_fn(r<-1?LOG_WARN:LOG_INFO, LD_DIR,
              "Unable to load consensus directory downloaded from "
-             "server '%s:%d'", conn->_base.address, conn->_base.port);
+             "server '%s:%d'. I'll try again soon.",
+             conn->_base.address, conn->_base.port);
       tor_free(body); tor_free(headers); tor_free(reason);
       networkstatus_consensus_download_failed(0);
       return -1;

Modified: tor/branches/tor-0_2_0-patches/src/or/networkstatus.c
===================================================================
--- tor/branches/tor-0_2_0-patches/src/or/networkstatus.c	2008-04-24 02:43:15 UTC (rev 14440)
+++ tor/branches/tor-0_2_0-patches/src/or/networkstatus.c	2008-04-24 05:17:58 UTC (rev 14441)
@@ -452,10 +452,10 @@
       });
     SMARTLIST_FOREACH(missing_authorities, trusted_dir_server_t *, ds,
       {
-        log(severity, LD_DIR, "Consensus does not include configured "
-            "authority '%s' at %s:%d (identity %s)",
-            ds->nickname, ds->address, (int)ds->dir_port,
-            hex_str(ds->v3_identity_digest, DIGEST_LEN));
+        log_info(LD_DIR, "Consensus does not include configured "
+                 "authority '%s' at %s:%d (identity %s)",
+                 ds->nickname, ds->address, (int)ds->dir_port,
+                 hex_str(ds->v3_identity_digest, DIGEST_LEN));
       });
     log(severity, LD_DIR,
         "%d unknown, %d missing key, %d good, %d bad, %d no signature, "
@@ -1403,10 +1403,8 @@
     if (r == -1) {
       /* Okay, so it _might_ be signed enough if we get more certificates. */
       if (!was_waiting_for_certs) {
-        /* XXX020 eventually downgrade this log severity, or make it so
-         * users know why they're being told. */
-        log_notice(LD_DIR, "Not enough certificates to check networkstatus "
-                   "consensus");
+        log_info(LD_DIR,
+                 "Not enough certificates to check networkstatus consensus");
       }
       if (!current_consensus ||
           c->valid_after > current_consensus->valid_after) {