[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] r17920: {tor} Bridge relays that had DirPort set to 0 would stop fetching (in tor/trunk: . src/or)



Author: arma
Date: 2009-01-05 13:54:11 -0500 (Mon, 05 Jan 2009)
New Revision: 17920

Modified:
   tor/trunk/ChangeLog
   tor/trunk/src/or/dirserv.c
Log:
Bridge relays that had DirPort set to 0 would stop fetching
descriptors shortly after startup, and then briefly resume
after a new bandwidth test and/or after publishing a new bridge
descriptor. Bridge users that try to bootstrap from them would
get a recent networkstatus but would get descriptors from up to
18 hours earlier, meaning most of the descriptors were obsolete
already. Reported by Tas; bugfix on 0.2.0.13-alpha.


Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2009-01-05 17:53:24 UTC (rev 17919)
+++ tor/trunk/ChangeLog	2009-01-05 18:54:11 UTC (rev 17920)
@@ -6,6 +6,13 @@
       practice it just means we spend many minutes trying directory
       mirrors that are long gone from the network. Helps bug 887 a bit;
       bugfix on 0.2.0.x.
+    - Bridge relays that had DirPort set to 0 would stop fetching
+      descriptors shortly after startup, and then briefly resume
+      after a new bandwidth test and/or after publishing a new bridge
+      descriptor. Bridge users that try to bootstrap from them would
+      get a recent networkstatus but would get descriptors from up to
+      18 hours earlier, meaning most of the descriptors were obsolete
+      already. Reported by Tas; bugfix on 0.2.0.13-alpha.
 
   o Minor features:
     - New controller event "clients_seen" to report a geoip-based summary

Modified: tor/trunk/src/or/dirserv.c
===================================================================
--- tor/trunk/src/or/dirserv.c	2009-01-05 17:53:24 UTC (rev 17919)
+++ tor/trunk/src/or/dirserv.c	2009-01-05 18:54:11 UTC (rev 17920)
@@ -1215,7 +1215,8 @@
 int
 directory_too_idle_to_fetch_descriptors(or_options_t *options, time_t now)
 {
-  return !options->DirPort && !options->FetchUselessDescriptors &&
+  return !directory_caches_dir_info(options) &&
+         !options->FetchUselessDescriptors &&
          rep_hist_circbuilding_dormant(now);
 }