[or-cvs] [tor/master] fix "Got a certificate for ?? that we already have"

[arma at torproject.org]

Author: Roger Dingledine <arma at torproject.org>
Date: Mon, 19 Apr 2010 18:35:57 -0400
Subject: fix "Got a certificate for ?? that we already have"
Commit: de4a49adc2bc7bc8a2dfe44d75a47a828056a52c

what's happening here is that we're fetching certs for obsolete
authorities -- probably legacy signers in this case. but try to
remain general in the log message.
---
 src/or/routerlist.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 6c97bd0..b4fb8d1 100644
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -185,15 +185,15 @@ trusted_dirs_load_certs_from_string(const char *contents, int from_store,
       log_info(LD_DIR, "Skipping %s certificate for %s that we "
                "already have.",
                from_store ? "cached" : "downloaded",
-               ds ? ds->nickname : "??");
+               ds ? ds->nickname : "an old or new authority");
 
       /* a duplicate on a download should be treated as a failure, since it
        * probably means we wanted a different secret key or we are trying to
        * replace an expired cert that has not in fact been updated. */
       if (!from_store) {
-        log_warn(LD_DIR, "Got a certificate for %s that we already have. "
-                 "Maybe they haven't updated it.  Waiting for a while.",
-                 ds ? ds->nickname : "??");
+        log_warn(LD_DIR, "Got a certificate for %s, but we already have it. "
+                 "Maybe they haven't updated it. Waiting for a while.",
+                 ds ? ds->nickname : "an old or new authority");
         authority_cert_dl_failed(cert->cache_info.identity_digest, 404);
       }
 
-- 
1.6.5