Browse Source

Stop declining to download microdescs with future published times.

This change is the only one necessary to allow future versions of
the microdescriptor consensus to replace every 'published' date with
e.g. 2038-01-01 00:00:00; this will save 50-75% in compressed
microdescriptor diff size, which is quite significant.

This commit is a minimal change for 0.2.9; future series will
reduce the use of the 'published' date even more.

Implements part of ticket 21642; implements part of proposal 275.
Nick Mathewson 7 years ago
parent
commit
85cf6dcba3
2 changed files with 9 additions and 5 deletions
  1. 9 0
      changes/prop275-minimal
  2. 0 5
      src/or/networkstatus.c

+ 9 - 0
changes/prop275-minimal

@@ -0,0 +1,9 @@
+  o Minor features (future-proofing):
+
+    - Tor no longer refuses to download microdescriptors or descriptors if
+      they are listed as "published in the future".  This change will
+      eventually allow us to stop listing meaningful "published" dates
+      in microdescriptor consensuses, and thereby allow us to reduce the
+      resources required to download consensus diffs by over 50%.
+      Implements part of ticket 21642; implements part of proposal 275.
+

+ 0 - 5
src/or/networkstatus.c

@@ -2360,11 +2360,6 @@ client_would_use_router(const routerstatus_t *rs, time_t now,
      * But, if we want to have a complete list, fetch it anyway. */
     return 0;
   }
-  if (rs->published_on + options->TestingEstimatedDescriptorPropagationTime
-      > now) {
-    /* Most caches probably don't have this descriptor yet. */
-    return 0;
-  }
   if (rs->published_on + OLD_ROUTER_DESC_MAX_AGE < now) {
     /* We'd drop it immediately for being too old. */
     return 0;