[or-cvs] r13337: Fix bug 597: stop telling people to email Tor-ops. Also give (in tor/trunk: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2008-01-30 13:32:30 -0500 (Wed, 30 Jan 2008)
New Revision: 13337

Modified:
   tor/trunk/
   tor/trunk/ChangeLog
   tor/trunk/src/or/networkstatus.c
Log:
 r17844 at catbus:  nickm | 2008-01-30 13:31:37 -0500
 Fix bug 597: stop telling people to email Tor-ops.  Also give a better suggestion when some other identity has been assigned the nickname we are using.



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r17844] on 8246c3cf-6607-4228-993b-4d95d33730f1

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-01-30 15:26:14 UTC (rev 13336)
+++ tor/trunk/ChangeLog	2008-01-30 18:32:30 UTC (rev 13337)
@@ -3,6 +3,8 @@
     - Directory caches now fetch certificates from all authorities
       listed in a networkstatus consensus, even when they do not
       recognize them.  Fixes bug 571.  Bugfix on 0.2.0.x.
+    - Stop recommending that every server operator send mail to tor-ops.
+      Resolves bug 597.  Bugfix on 0.1.2.x.
 
 
 Changes in version 0.2.0.18-alpha - 2008-01-25

Modified: tor/trunk/src/or/networkstatus.c
===================================================================
--- tor/trunk/src/or/networkstatus.c	2008-01-30 15:26:14 UTC (rev 13336)
+++ tor/trunk/src/or/networkstatus.c	2008-01-30 18:32:30 UTC (rev 13337)
@@ -1527,6 +1527,7 @@
   routerinfo_t *me;
   routerlist_t *rl = router_get_routerlist();
   networkstatus_vote_t *consensus = networkstatus_get_live_consensus(now);
+  or_options_t *options = get_options();
 
   if (networkstatus_v2_list_has_changed)
     download_status_map_update_from_v2_networkstatus();
@@ -1554,15 +1555,14 @@
                "Are you misconfigured?");
       have_warned_about_invalid_status = 1;
     } else if (rs->is_unnamed) {
-      /* XXXX020 this isn't a useful warning. */
+      /* Maybe upgrade this to notice? XXXX020 */
       log_info(LD_GENERAL,  "The directory have assigned the nickname "
-               "you're using to a different identity.");
+               "you're using (%s) to a different identity; you may want to "
+               "choose a different nickname.", options->Nickname);
       have_warned_about_invalid_status = 1;
     } else if (!rs->is_named) {
-      /*XXXX020 this isn't a correct warning. */
-      log_info(LD_GENERAL,  "The directory authorities do not recognize "
-               "your nickname. Please consider sending your "
-               "nickname and identity fingerprint to the tor-ops.");
+      log_debug(LD_GENERAL, "The directory authorities do not currently "
+                "recognize your nickname.");
       have_warned_about_invalid_status = 1;
     }
   }