[or-cvs] r14419: Backport: Periodically launch requests for server/authority. (in tor/branches/tor-0_2_0-patches: . src/or)

[nickm@xxxxxxxx]

Author: nickm
Date: 2008-04-22 17:11:00 -0400 (Tue, 22 Apr 2008)
New Revision: 14419

Modified:
   tor/branches/tor-0_2_0-patches/
   tor/branches/tor-0_2_0-patches/ChangeLog
   tor/branches/tor-0_2_0-patches/src/or/routerlist.c
Log:
 r15288@tombo:  nickm | 2008-04-22 17:09:36 -0400
 Backport: Periodically launch requests for server/authority.z when it might help us learn our IP.  Fix for bug 652.



Property changes on: tor/branches/tor-0_2_0-patches
___________________________________________________________________
 svk:merge ticket from /tor/020 [r15288] on 49666b30-7950-49c5-bedf-9dc8f3168102

Modified: tor/branches/tor-0_2_0-patches/ChangeLog
===================================================================
--- tor/branches/tor-0_2_0-patches/ChangeLog	2008-04-22 21:10:41 UTC (rev 14418)
+++ tor/branches/tor-0_2_0-patches/ChangeLog	2008-04-22 21:11:00 UTC (rev 14419)
@@ -36,6 +36,8 @@
       Patch from mwenge.  Fixes bug 646.
     - Correctly notify one-hop connections when a circuit build has
       failed.  Possible fix for bug 669.  Found by lodger.
+    - Detect address changes correctly on non-directory mirror
+      servers.  Fix for bug 652.
 
   o Minor features (security):
     - Reject requests for reverse-dns lookup of names in a private

Modified: tor/branches/tor-0_2_0-patches/src/or/routerlist.c
===================================================================
--- tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-04-22 21:10:41 UTC (rev 14418)
+++ tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-04-22 21:11:00 UTC (rev 14419)
@@ -3962,17 +3962,37 @@
   smartlist_free(no_longer_old);
 }
 
+/** How often should we launch a server/authority request to be sure of getting
+ * a guess for our IP? */
+/*XXXX021 this info should come from netinfo cells or something, or we should
+ * do this only when we aren't seeing incoming data. see bug 652. */
+#define DUMMY_DOWNLOAD_INTERVAL (20*60)
+
 /** Launch downloads for router status as needed. */
 void
 update_router_descriptor_downloads(time_t now)
 {
   or_options_t *options = get_options();
+  static time_t last_dummy_download = 0;
   if (should_delay_dir_fetches(options))
     return;
   if (directory_fetches_dir_info_early(options)) {
     update_router_descriptor_cache_downloads_v2(now);
   }
   update_consensus_router_descriptor_downloads(now);
+
+  /* XXXX021 we could be smarter here; see notes on bug 652. */
+  /* If we're a server that doesn't have a configured address, we rely on
+   * directory fetches to learn when our address changes.  So if we haven't
+   * tried to get any routerdescs in a long time, try a dummy fetch now. */
+  if (!options->Address &&
+      server_mode(options) &&
+      last_routerdesc_download_attempted + DUMMY_DOWNLOAD_INTERVAL < now &&
+      last_dummy_download + DUMMY_DOWNLOAD_INTERVAL < now) {
+    last_dummy_download = now;
+    directory_get_from_dirserver(DIR_PURPOSE_FETCH_SERVERDESC,
+                                 ROUTER_PURPOSE_GENERAL, "authority.z", 1);
+  }
 }
 
 /** Launch extrainfo downloads as needed. */