[or-cvs] r12005: When a networkstatus consensus download fails, do not wait 6 (in tor/trunk: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2007-10-17 17:26:22 -0400 (Wed, 17 Oct 2007)
New Revision: 12005

Modified:
   tor/trunk/
   tor/trunk/ChangeLog
   tor/trunk/src/or/main.c
   tor/trunk/src/or/networkstatus.c
Log:
 r15887 at catbus:  nickm | 2007-10-17 17:25:32 -0400
 When a networkstatus consensus download fails, do not wait 60 seconds to decide whether to retry. (Also, log the time at which we'll try to replace the current networkstatus.)



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

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2007-10-17 21:26:19 UTC (rev 12004)
+++ tor/trunk/ChangeLog	2007-10-17 21:26:22 UTC (rev 12005)
@@ -38,6 +38,8 @@
 
   o Minor bugfixes (v3 directory protocol)
     - Delete unverified-consensus when the real consensus is set.
+    - Consider retrying a consensus networkstatus fetch immediately after one
+      fails: don't wait 60 seconds to notice.
 
   o Minor bugfixes (controller):
     - Don't reset trusted dir server list when we set a configuration option.

Modified: tor/trunk/src/or/main.c
===================================================================
--- tor/trunk/src/or/main.c	2007-10-17 21:26:19 UTC (rev 12004)
+++ tor/trunk/src/or/main.c	2007-10-17 21:26:22 UTC (rev 12005)
@@ -997,9 +997,6 @@
     networkstatus_v2_list_clean(now);
     /* Remove dead routers. */
     routerlist_remove_old_routers();
-#if 0
-    networkstatus_v2_list_update_recent(now);
-#endif
 
     /* Also, once per minute, check whether we want to download any
      * networkstatus documents.

Modified: tor/trunk/src/or/networkstatus.c
===================================================================
--- tor/trunk/src/or/networkstatus.c	2007-10-17 21:26:19 UTC (rev 12004)
+++ tor/trunk/src/or/networkstatus.c	2007-10-17 21:26:22 UTC (rev 12005)
@@ -728,6 +728,8 @@
 update_consensus_networkstatus_downloads(time_t now)
 {
   or_options_t *options = get_options();
+  if (!networkstatus_get_live_consensus(now))
+    time_to_download_next_consensus = now;
   if (time_to_download_next_consensus > now)
     return;
   if (authdir_mode_v3(options))
@@ -747,6 +749,8 @@
 networkstatus_consensus_download_failed(int status_code)
 {
   download_status_failed(&consensus_dl_status, status_code);
+  /* Retry immediately, if appropriate. */
+  update_consensus_networkstatus_downloads(time(NULL));
 }
 
 /** DOCDOC */
@@ -755,8 +759,8 @@
 {
   or_options_t *options = get_options();
   /* XXXX020 call this when DirPort switches on or off. NMNM */
-  if (current_consensus) {
-    const networkstatus_vote_t *c = current_consensus;
+  networkstatus_vote_t *c = networkstatus_get_live_consensus(now);
+  if (c) {
     time_t start;
     long interval;
     if (dirserver_mode(options)) {
@@ -772,9 +776,16 @@
       interval = 1;
     tor_assert(start+interval < c->valid_until);
     time_to_download_next_consensus = start + crypto_rand_int(interval);
+    {
+      char tbuf[ISO_TIME_LEN+1];
+      format_local_iso_time(tbuf, time_to_download_next_consensus);
+      log_info(LD_DIR, "Have a live consensus; fetching next one at %s.",tbuf);
+    }
   } else {
     time_to_download_next_consensus = now;
+    log_info(LD_DIR, "No live consensus; we should fetch one immediately.");
   }
+
 }
 
 /** Return 1 if there's a reason we shouldn't try any directory