commit c8b855082bdb4ea5f9006e6ce5ce673509643a5c Author: Nick Mathewson nickm@torproject.org Date: Fri Feb 10 12:01:56 2012 -0500
Downgrade "missing a certificate" from notice to info
It was apparently getting mistaken for a problem, even though it was at notice.
Fixes 5067; fix on 0.2.0.10-alpha. --- changes/bug5067 | 5 +++++ src/or/routerlist.c | 6 +++--- 2 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/changes/bug5067 b/changes/bug5067 new file mode 100644 index 0000000..d94b921 --- /dev/null +++ b/changes/bug5067 @@ -0,0 +1,5 @@ + o Minor bugfixes (usability): + - Downgrade the "We're missing a certificate" message from notice + to info: people kept mistaking it for a real problem, whereas it + is only a problem when we are failing to bootstrap. Fixes bug + 5067; bugfix on 0.2.10-alpha. diff --git a/src/or/routerlist.c b/src/or/routerlist.c index f8df089..1edf651 100644 --- a/src/or/routerlist.c +++ b/src/or/routerlist.c @@ -504,9 +504,9 @@ authority_certs_fetch_missing(networkstatus_t *status, time_t now) if (download_status_is_ready(&cl->dl_status, now, MAX_CERT_DL_FAILURES) && !digestmap_get(pending, voter->identity_digest)) { - log_notice(LD_DIR, "We're missing a certificate from authority " - "with signing key %s: launching request.", - hex_str(sig->signing_key_digest, DIGEST_LEN)); + log_info(LD_DIR, "We're missing a certificate from authority " + "with signing key %s: launching request.", + hex_str(sig->signing_key_digest, DIGEST_LEN)); smartlist_add(missing_digests, sig->identity_digest); } } SMARTLIST_FOREACH_END(sig);